Skip to content

File path tweaks in UI - #2264

Merged
appleboy merged 1 commit into
go-gitea:masterfrom
silverwind:path-ellipsis
Aug 6, 2017
Merged

File path tweaks in UI#2264
appleboy merged 1 commit into
go-gitea:masterfrom
silverwind:path-ellipsis

Conversation

@silverwind

@silverwind silverwind commented Aug 5, 2017

Copy link
Copy Markdown
Member
  • Adjust file path to wrap at 30 chars instead of 25/15.
  • Allow text selection of path. No idea why Semantic UI blocks selection on those, so I overrode it.
  • Turn a few <div>s into <span>s so that copied text from the path does not include newlines.

Before

After

@appleboy appleboy added type/enhancement An improvement of existing functionality topic/ui Change the appearance of the Gitea UI labels Aug 5, 2017
@appleboy appleboy added this to the 1.2.0 milestone Aug 5, 2017
@Bwko

Bwko commented Aug 5, 2017

Copy link
Copy Markdown
Member

LGTM

@tboerger tboerger added the lgtm/need 1 This PR needs approval from one additional maintainer to be merged. label Aug 5, 2017
@lafriks

lafriks commented Aug 5, 2017

Copy link
Copy Markdown
Member

LGTM

@tboerger tboerger added lgtm/done This PR has enough approvals to get merged. There are no important open reservations anymore. and removed lgtm/need 1 This PR needs approval from one additional maintainer to be merged. labels Aug 5, 2017
@appleboy
appleboy merged commit 54381f4 into go-gitea:master Aug 6, 2017
@silverwind
silverwind deleted the path-ellipsis branch August 6, 2017 05:36
@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

lgtm/done This PR has enough approvals to get merged. There are no important open reservations anymore. topic/ui Change the appearance of the Gitea UI type/enhancement An improvement of existing functionality

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants