Opened 3 months ago
Closed 2 months ago
#64962 closed defect (bug) (fixed)
Install screens: “Log In” and "Support link" use old color
| Reported by: |
|
Owned by: |
|
|---|---|---|---|
| Milestone: | 7.0 | Priority: | normal |
| Severity: | normal | Version: | 7.0 |
| Component: | Administration | Keywords: | admin-reskin has-patch has-screenshots dev-reviewed |
| Focuses: | ui, css | Cc: |
Description
In the WordPress install flow, links such as “Read the support article on wp-config.php” and “Log In” are using the outdated link color instead of the updated admin color.
Attachments (3)
Change History (27)
This ticket was mentioned in PR #11373 on WordPress/wordpress-develop by @hbhalodia.
3 months ago
#3
- Keywords has-patch added
Trac ticket: https://core.trac.wordpress.org/ticket/64962
## Description
- Add CSS variable for install.css changes.
- Add color hex code for
wp_diefunctions.php code, as the variable was not added before that it executed wp_die. - Also added fix for
setup-config.phpif config already exists and related anchors. See before and after Screenshots.
| Before | After |
|---|---|
| | |
## Use of AI Tools
- None
@audrasjb commented on PR #11373:
3 months ago
#5
@audrasjb commented on PR #11373:
3 months ago
#6
Thanks for the PR, this looks good to me 👍
#7
@
3 months ago
- Component changed from General to Administration
- Keywords commit has-screenshots added
- Version set to trunk
Thanks for the PR, this is ready to ship.
#9
@
3 months ago
I'm separating this in two commits, as the two patches are fixing the issue in very different locations and with a different implementation.
#10
@
3 months ago
- Keywords changes-requested added; commit removed
Wait, @hbhalodia are you sure these are the correct colors?
By the way we'll also want to update :focus colors and the other related styles (like the border/border-radius of the outline).
Removing commit for now.
This ticket was mentioned in Slack in #core by audrasjb. View the logs.
2 months ago
#12
@
2 months ago
As per today's bug scrub:
I'm not 100% sure the proposed PR uses the good color values. I pinged the PR author but another check is welcome.
Invoking @wildworks or @sabernardt, just in case you can help to confirm the issue or not?
#13
@
2 months ago
I tested the changes locally using the provided updated code, and the updated link colors look consistent with the new admin design system. The “Log In” and support article links now match other admin link styles.
From my testing, the color values appear correct and align well with the intended UI refresh. Overall, the update looks good to me.
#14
@
2 months ago
tested the changes locally with the updated code, and the link colors are now consistent with the revised admin design system. The “Log In” and support article links align well with other admin link styles, improving overall visual consistency.
From my review, the color values look accurate, and the update meets the intended UI expectations.
#15
@
2 months ago
Tested locally — link colors now match the new admin design system. “Log In” and support links are consistent with other admin styles. Overall, looks good.
#16
@
2 months ago
Verified the updates in a local environment, and the revised link colors integrate well with the refreshed admin UI. The Log In and support links now follow a consistent styling pattern, enhancing visual clarity across the interface.
#17
@
2 months ago
The updates were successfully validated in a local environment, with the revised link colors aligning seamlessly with the refreshed admin UI. The Log In and support links now adhere to a consistent styling pattern, improving visual coherence and enhancing overall interface clarity.
#18
@
2 months ago
Tested the changes locally and confirmed that the updated link colors align with the revised admin design system.
#19
@
2 months ago
PR 11373 needs to be updated. The correct color is not being used in some parts of the code. Please refer to this discussion.
cc @hbhalodia
#22
@
2 months ago
- Keywords dev-feedback added
- Resolution fixed deleted
- Status changed from closed to reopened
🐞 Bug occurs
PHP: 8.1
WordPress Version: 7.0-RC2