In #1608878-30: Add CTools dropbutton to core, Dave Reid pointed out that the dropbutton could be hard to use on small screens.

I think adding a bunch of padding in a media query is a good idea, similar to the menu links in Bartik.

Comments

dead_arm’s picture

dead_arm
she/her
Primary language English
commented
Assigned: Unassigned » dead_arm

Assigning to myself to work on after the regular dropbutton is committed.

webchick’s picture

webchick
she/they
Primary language English
Location Vancouver 🇨🇦
commented
Status: Postponed » Postponed (maintainer needs more info)

Is this still relevant? It seems like the dropbutton patch got the shit kicked out of it before commit. :)

kiphaas7’s picture

kiphaas7 commented
Status: Postponed (maintainer needs more info) » Active

It is. Each drop button gets at least 3 classes added via js and an element added (the toggle). If that is going to happen at 50 or more drop buttons' its going to have an impact for mobile.

I'd propose to move all classes to php, change the CSS to .js .class so it (still) only runs when is js enabled. Possibly also add the button already via php and hide it by default.

kiphaas7’s picture

kiphaas7 commented
Status: Active » Postponed (maintainer needs more info)

Ups, comment should be in #1799498: Improve dropbutton. Sorry for the noise.

dead_arm’s picture

dead_arm
she/her
Primary language English
commented
Assigned: dead_arm » Unassigned
nod_’s picture

nod_
Primary language French
Location Lille
commented
Issue summary: View changes
Status: Postponed (maintainer needs more info) » Closed (duplicate)

Feel free to reopen if still too small.