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

playlist editor dense mode #14

Open
echarlie opened this issue Jul 3, 2020 · 2 comments
Open

playlist editor dense mode #14

echarlie opened this issue Jul 3, 2020 · 2 comments
Labels
enhancement New feature or request needs-user-feedback We should either test with users or ask users if they would like this.

Comments

@echarlie
Copy link
Member

echarlie commented Jul 3, 2020

the playlist editor is hard to use because it is not dense. we should either make it more dense, or make a checkbox to set the density to high.

@mutantmonkey mutantmonkey added the enhancement New feature or request label Jul 5, 2020
@mutantmonkey
Copy link
Member

It is worth noting that we are already using the table-condensed Bootstrap class. One of the things that makes the editor "not dense" is the height of the buttons (which we're already using btn-sm for as well). We may need to do something like put a negative margin on the button cell or manually make the buttons even smaller to do what you're asking for here.

@echarlie echarlie added the needs-user-feedback We should either test with users or ask users if they would like this. label Jul 21, 2020
@mutantmonkey
Copy link
Member

mutantmonkey commented Jul 27, 2020

I found out that table-condensed was apparently renamed to table-sm, so that class isn't doing anything after all. That being said, the buttons are still going to be taller than the text, so we may want to change padding on that cell.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request needs-user-feedback We should either test with users or ask users if they would like this.
Projects
None yet
Development

No branches or pull requests

2 participants