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

Hide SHA-Hash column #126

Closed
Herr-Sepp opened this issue Mar 20, 2020 · 6 comments · Fixed by #131
Closed

Hide SHA-Hash column #126

Herr-Sepp opened this issue Mar 20, 2020 · 6 comments · Fixed by #131
Assignees

Comments

@Herr-Sepp
Copy link

Hi thanks for your great UI.

On your gif the column for the sha-hash is hidden and if i understand it correctly it will only show if i hover over with the cursor.

image

But on my instance and even the demo the column is always shown and makes the table difficult to read because the rows are so wide.
image

Is this a bug or how can I achieve to hide the sha-column by default?

@Herr-Sepp Herr-Sepp changed the title Hide sha-hash Hide SHA-Hash column Mar 20, 2020
@Joxit
Copy link
Owner

Joxit commented Mar 21, 2020

Hi

Sorry, this is not a bug, the gif was made at version 1.3.0 and the latest UI version is 1.4.8...
The feature has been added in 1.4.0 with #102
We did some tests with different windows sizes and the digest should be hidden at 710px.

I should update the GIF 😅

@Herr-Sepp
Copy link
Author

Any chance to give us a flag or something to hide this column?
I dont need it and when a tag is rellay long, i got a scrollbar on the bottom of the table even on big screens.

@Joxit
Copy link
Owner

Joxit commented Mar 21, 2020

Hum... Yes I think it's possible

What is the spec of your screen ? In px ? Retina supported ?

@Herr-Sepp
Copy link
Author

image

27" 2560X1440 (windows scaling 100%)

But this happens on any screen, because the table will be wider then the hard limit of 1440px with Content Digest and Tag column.

@Joxit
Copy link
Owner

Joxit commented Apr 28, 2020

Oh! Ok, this is because the tag name is longer than expected ! I understand, thanks for the intel 🙂

Joxit added a commit that referenced this issue May 25, 2020
@Joxit Joxit self-assigned this May 30, 2020
@Joxit
Copy link
Owner

Joxit commented May 30, 2020

Hi @Herr-Sepp, I did a PR for this issue, you can hide the digest with SHOW_CONTENT_DIGEST=false.
But I also fixed the digest length when a tag is too long 😄

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Development

Successfully merging a pull request may close this issue.

2 participants