Early Bird Registration for DrupalCon Portland 2024 is open! Register by 23:59 PST on 31 March 2024, to get $100 off your ticket.
We have a hover state for the green "Update this issue" button (link) in the metadata table, but the actual buttons on forms provide no visual indication that they're being hovered over. My cursor changes, which is helpful (thanks browser), but I don't understand why these remain so static on hover.
Comment | File | Size | Author |
---|---|---|---|
#10 | bluecheese-button-hover-2127423-10.patch | 827 bytes | DyanneNova |
#3 | bluecheese-button-hover-2127423.patch | 459 bytes | LewisNyman |
Comments
Comment #1
eliza411 CreditAttribution: eliza411 commentedComment #2
mgiffordSo the hover state we use in the sidbar is:
Which can just be added in like this to the submit button:
I guess we'd just need a patch for this.
Comment #3
LewisNyman CreditAttribution: LewisNyman commentedAs we're playing with colours we need to use a colour variable in Sass. I guess the .issue-button CSS isn't in Bluecheese? We have button styling declared in about 4 different places....
Comment #4
mgiffordThat looks good. It applies nicely to bluecheese. I just don't have the capacity to compile the sass files.
Comment #5
Bojhan CreditAttribution: Bojhan commentedScreenshot?
Comment #6
tvn CreditAttribution: tvn commentedThanks for working on this! Please provide a screenshot and/or link to the dev site where we can see the patch in action.
Comment #7
drummThis won't work because this style:
@include background-image(linear-gradient(top, $button-background 50%, $lime-green 50%));
Always takes precedence over a solid background color. The hover state needs the same two-tone effect. (Which can't be seen on the unbranded public theme, it is all the same green.)
This should also remove the nearby
which is replaced by this patch.
The
.issue-button
styles are in project_issue module's CSS, which should be cleaned up as we notice things.Comment #8
LewisNyman CreditAttribution: LewisNyman commentedOk, I'll try this again later when I can develop against the branded theme.
Comment #9
drummThe public theme should have more different shades of green so we can work on this. (Or, should the smaller buttons actually be two-tone?)
Comment #10
DyanneNovaHere's a bzr patch that takes the background image into account for the branded theme.
Comment #11
DyanneNovaComment #13
drummLooks good!
Comment #14
drummNow deployed.