Opened 5 years ago
Last modified 18 months ago
#48553 assigned defect (bug)
"Admin Color Scheme" should not be saved before hitting the "Update Profile" button in profile page
Reported by: | mukto90 | Owned by: | |
---|---|---|---|
Milestone: | Awaiting Review | Priority: | normal |
Severity: | normal | Version: | 3.8 |
Component: | Users | Keywords: | has-screenshots has-patch has-testing-info |
Focuses: | ui, administration | Cc: |
Description
On the profile page (/wp-admin/profile.php) when you click on a color scheme, it instantly updates (via AJAX) in the user meta and changes the color.
I think this should not happen. User meta should be updated only after they hit the "Update Profile" button. It's OK to show the color (preview) as soon as they click an option, but storing the value in the database should be updated along with other info after clicking the save button.
Attachments (2)
Change History (6)
#2
@
5 years ago
I think previewing it and not saving it would be more confusing. Users would see the color scheme they want, then navigate away from the page where it would be reverted back to the previous value.
There is probably some discussion about this from back when this was first added, but I can’t find it right now.
#4
@
18 months ago
- Keywords has-testing-info added
Test Report
Patch tested: https://core.trac.wordpress.org/attachment/ticket/48553/48553.diff
Environment
- OS: macOS 12.4
- Web Server: Nginx
- PHP: 7.4.33
- WordPress: 6.3-alpha-55505-src
- Browser: Chrome
- Theme: Twenty Twenty-two, Twenty-nineteen
ScreenCast
Before applying the patch: https://www.loom.com/share/fee1873fe6ec41e6bb105f59e5b26581
After applying the patch: https://www.loom.com/share/8610ef1921bb4f1b880fa2dd69d5194d
Notes
After applying the patch, the user needs to click Update profile to change the color scheme.
Actual Results
- The patch is working as expected✅
I've attached a patch