Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Multiple-Line commit message #2939

Closed
2 tasks done
sinnlosername opened this issue Nov 20, 2017 · 10 comments
Closed
2 tasks done

Multiple-Line commit message #2939

sinnlosername opened this issue Nov 20, 2017 · 10 comments
Labels
type/enhancement An improvement of existing functionality
Milestone

Comments

@sinnlosername
Copy link
Contributor

  • Gitea version (or commit ref): 1.3.0+rc1
  • Git version: 2.7.4
  • Operating system: Ubuntu 16.04
  • Database (use [x]):
    • MySQL
  • Can you reproduce the bug at https://try.gitea.io:
    • No: Didn't find a matching commit

Description

Only the first line of a commit message is shown.

Screenshots

How it looks in gitea:
Commit history -> https://i.imgur.com/oVDhmAP.png
Commit view -> https://i.imgur.com/lXAfOxV.png

The whole commit message (from a discord webhook):
https://i.imgur.com/9BeVaIS.png

@lunny lunny added the type/bug label Nov 20, 2017
@lunny lunny added this to the 1.x.x milestone Nov 20, 2017
@makarchuk
Copy link
Contributor

What is expected behaviour here?
It would be really strange to show all lines of commit message, it can be really long

@lafriks
Copy link
Member

lafriks commented Nov 20, 2017

I think full message should be shown only when viewing commit diff page

@lafriks
Copy link
Member

lafriks commented Nov 20, 2017

And also collapsed by default and expandable when clicked on

@sinnlosername
Copy link
Contributor Author

You could also collaps it in the commit list, like github does.

@davydov-vyacheslav
Copy link

@lafriks , maybe another way is to make tooltip with multiline information over singlelined current behavior. At least, this way affects UI changes less

@lafriks
Copy link
Member

lafriks commented Nov 20, 2017

@davydov-vyacheslav it's also an option. Only disadvantage of that is you can not copy text from it if you need it

@makarchuk
Copy link
Contributor

I implemented @davydov-vyacheslav's idea, you're welcome to take a look and criticize me if I did anything wrong.

@sinnlosername
Copy link
Contributor Author

I think something similar to github would be a good idea, because as @lafriks said, it's not possible to copy something. Also, it's not that good to read.

@lafriks lafriks added type/enhancement An improvement of existing functionality and removed backport/v1.3 type/bug labels Nov 28, 2017
@sondr3
Copy link
Contributor

sondr3 commented Nov 28, 2017

My pull request for this at #2980 is now complete (I hope), if anyone would like to test it I'd greatly appreciate it.

@lafriks lafriks modified the milestones: 1.x.x, 1.4.0 Nov 28, 2017
@lunny
Copy link
Member

lunny commented Dec 12, 2017

resolved by #2980

@lunny lunny closed this as completed Dec 12, 2017
@go-gitea go-gitea locked and limited conversation to collaborators Nov 23, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
type/enhancement An improvement of existing functionality
Projects
None yet
Development

Successfully merging a pull request may close this issue.

6 participants