-
Notifications
You must be signed in to change notification settings - Fork 7
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
Feat/create more space #324
Conversation
…emoving unnecessary white-space
# Conflicts: # cypress/e2e/debug_aboutOpenedReports.cy.js # cypress/e2e/testtab.cy.js
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Im very curious what buildTarget
is and why you changed to that instead of browserTarget
. Ive never seen this before
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Currently spacing 0 is the default but I would change that to 1 as the default. Also only showing numbers doesnt really explain what the unit of the spacing is, that might be nice to add |
Select all and deselect all buttons have been replaced with a checkbox in the table header.
Padding on table rows have been minimized to create more space.
Metadata table visible under the editor has been moved above it and can be shown or hidden with a button.
Table header names and column values have been shortened, hovering on the table header will still show the full name.
Table row spacing can be altered in settings, font size is also changed with the spacing.