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

Too much whitespace in dropdowns #2

Closed
joshangell opened this issue Jul 4, 2018 · 6 comments
Closed

Too much whitespace in dropdowns #2

joshangell opened this issue Jul 4, 2018 · 6 comments
Assignees

Comments

@joshangell
Copy link
Contributor

joshangell commented Jul 4, 2018

From angell-co/Pimp-My-Matrix#25 by @bossanova808


I still think the whitespace is a lot too much though, personally. Or possibly some sort of nested organisation would help too...once you have a bunch of types, the drop down is still very large

@joshangell
Copy link
Contributor Author

Thanks for this Jeremy, I will see if we can tighten up the list or perhaps use some kind of toggle and collapse all the groups - for now I’m just using the native button list styles.

@joshangell joshangell self-assigned this Jul 4, 2018
@joshangell
Copy link
Contributor Author

@bossanova808 what about this:
image

joshangell added a commit that referenced this issue Jul 4, 2018
@bossanova808
Copy link

Hey Josh - well I certainly like it. I guess you might want other's opinions of course! And I guess it would be configurable?

That would certainly help us, at least - we have a pretty full on page builder going on (as you can see!), and I think this would make the editing process noticeably better

@joshangell
Copy link
Contributor Author

Yeah I think it's useful, having played with it a bit now. I'll add it as a configurable thing which you can turn on per Matrix field from a config file, does that sound good?

@bossanova808
Copy link

Sounds awesome!

@joshangell
Copy link
Contributor Author

OK, this is done in 8a89fd8

joshangell added a commit that referenced this issue Jul 10, 2018
- Added the `nestedSettings` config setting to allow a new, nested settings menu format ([#2](#2))
- Added the Craft 2 > 3 migration

- Fixed an issue where switching the entry type didn’t update the UI - requires Craft v3.0.15 or greater ([#4](#4))
- Fixed a missing translation group key ([#5](#5))

* tag '3.0.0':
  Changelog for 3.0.0
  Readme tweaks
  Require version 3.0.15 of Craft for the entry type switcher
  Temporarily downgrade the required craft cms version
  Fixed a missing translation group key.
  Prep for release, pending craftcms/cms:3.0.15 or similar.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants