Opened 7 weeks ago
Closed 4 weeks 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: | trunk |
| 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.
7 weeks 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:
7 weeks ago
#5
@audrasjb commented on PR #11373:
7 weeks ago
#6
Thanks for the PR, this looks good to me 👍
#7
@
7 weeks 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
@
7 weeks 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
@
7 weeks 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.
7 weeks ago
#12
@
7 weeks 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
@
6 weeks 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
@
5 weeks 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
@
5 weeks 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
@
5 weeks 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
@
5 weeks 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
@
5 weeks ago
Tested the changes locally and confirmed that the updated link colors align with the revised admin design system.
#19
@
5 weeks 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
@
5 weeks ago
- Keywords dev-feedback added
- Resolution fixed deleted
- Status changed from closed to reopened
🐞 Bug occurs
PHP: 8.1
WordPress Version: 7.0-RC2