This is a follow-up issue to #1831894: Users miss "save" button and can't distinguish "editable" and "preview" areas addressing the "Users miss the save button" portion of the previous task. Comment #7 recommended that there be two separate issues.
Pertinent details:
Posted by jessebeach on May 19, 2013 at 4:09pm
xjm, tim.plunkett, dwehner, jessebeach and dmistry discussed and concluded that the Save button will be placed in the bottom left, following the standard Drupal pattern. The save button in the top right will be removed.
#18
Posted by tim.plunkett on May 19, 2013 at 4:18pm
In addition, the Save button will be persistent. Currently it is only visible if the view has been modified.
Comment | File | Size | Author |
---|---|---|---|
#3 | Screen Shot 2013-05-24 at 8.40.34 PM.png | 12.1 KB | dead_arm |
#2 | views-2003440-change_location_save_button_style_yes-2.patch | 1.43 KB | iStryker |
#2 | views-2003440-change_location_save_button_style_no-2.patch | 1.31 KB | iStryker |
views-save-default_0.png | 152.67 KB | dead_arm |
Comments
Comment #1
iStryker CreditAttribution: iStryker commentedI'll take a crack at this
Comment #2
iStryker CreditAttribution: iStryker commentedAttached is 2 patches. One without style and one with a little amount of style. @dead_arm does this need to be styled like the image.
Patch removes actionsElement protected function as it serves no purpose anymore. Before it had 2 purposes. One to put the actions buttons above the display (weight = 0) and the other to hide the button if no changes have been made.
Comment #3
dead_arm@iStryker I asked Jessebeach earlier, and since we are following the Drupal 8 save button pattern by moving it below, we should use the same "big blue" button style styling that is in other places. The png in the issue summary is from the original issue and I think it predates the current save button styling.
Screenshot from node/add/article
Comment #4
dead_armActually the CSS code is already in place for the primary button to be blue, so there is a bug. Self-assigning this issue to resolve bug. Will follow-up shortly.
Comment #5
dead_armFollow-up issue for the button style regression #2003908: Regression of primary button styling
Comment #6
dead_arm#2: views-2003440-change_location_save_button_style_no-2.patch queued for re-testing.
Comment #7
dead_armviews-2003440-change_location_save_button_style_no-2.patch from #2 moves the buttons, and the button styling will be taken care of in #2003908: Regression of primary button styling
Comment #9
iStryker CreditAttribution: iStryker commented#2: views-2003440-change_location_save_button_style_no-2.patch queued for re-testing.
Comment #10
dead_armThe second patch of #2 (views-2003440-change_location_save_button_style_no-2.patch) finally got through testing and looks good!
Comment #11
tstoecklerI'm not sure, as I'm not much of a Views expert, but I assume there is some JS code that currently makes the Save button appear as soon as something has changed. That should be removed. Setting to "needs review" as I'm not really sure.
Comment #12
tim.plunkettThat's what this was.
Comment #13
iStryker CreditAttribution: iStryker commentedThere is JavaScript that toggles the js-hide class. This is still needed for the warning message about the view being change and requires to be saved
Comment #14
tstoecklerD'oh, sorry for the noise.
Comment #15
Dries CreditAttribution: Dries commentedCommitted to 8.x. Thanks.
Comment #17
dead_armComment #18
Gábor HojtsyThanks, removing from UX sprint now.