Skip to content
This repository has been archived by the owner on Jul 25, 2018. It is now read-only.

Checksum field for attachments is too small #151

Closed
mcjaeger opened this issue Apr 20, 2016 · 3 comments · Fixed by #216
Closed

Checksum field for attachments is too small #151

mcjaeger opened this issue Apr 20, 2016 · 3 comments · Fixed by #216
Assignees
Milestone

Comments

@mcjaeger
Copy link
Contributor

Somehow, the screen layout can be easily set into a mode where the checksum is truncatedly displayed. It would be good to have the complete checksum in most cases - for super small screens of course not, but checksum is truncated at fixed width, regardless of the screen width (for example, truncated at 1920 width) (tested with IE11)

@mcjaeger mcjaeger added this to the 1.4 milestone Apr 20, 2016
@mcjaeger mcjaeger added the bug label May 3, 2016
@mcjaeger mcjaeger modified the milestones: 1.5, 1.4 May 3, 2016
@mcjaeger
Copy link
Contributor Author

mcjaeger commented May 3, 2016

is also in Firefox (truncated always) please change also display as plain text (like label style) and not input test field

@maxhbr maxhbr self-assigned this Aug 31, 2016
@maxhbr
Copy link
Member

maxhbr commented Aug 31, 2016

The simple change input -> span introduces some ugliness when the browser width is not wide enougth (at 1200px the hash starts to be spread on multiple lines, 1650px are necessary to have the hash in the same line as the label) but might be acceptable:

tmp

@maxhbr
Copy link
Member

maxhbr commented Aug 31, 2016

By using word-break: break-all; one is able to obtain the following more pleasing behaviour:
tmp

maxhbr added a commit that referenced this issue Aug 31, 2016
... instead of `<input>`

closes #151
maxhbr added a commit that referenced this issue Aug 31, 2016
... instead of `<input>`

closes #151
mcjaeger added a commit that referenced this issue Aug 31, 2016
fix(#151): display hash with `<span>` instead of `<input>`
tested-by:[email protected]
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants