Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Make the styles not broken in 6.2 #175

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

jazzsequence
Copy link

The .welcome-panel class gets some new styling in 6.2 which makes the text virtually unreadable (black on dark blue). This PR flips the colors and adds some padding. Have not tested to see if these styles are enqueued globally but I don't think they are (looks like they're just here), but if this does affect things outside the importing page, a class could be added to the welcome-panel div wrapper to specify that it's a wp-import welcome-panel and then these CSS rules could be applied to wp-import.welcome-panel or just wp-import.

The `.welcome-panel` class gets some new styling in 6.2 which makes the text virtually unreadable (black on dark blue). This PR flips the colors and adds some padding. Have not tested to see if these styles are enqueued globally but I don't _think_ they are (looks like they're just [here](https://github.com/humanmade/WordPress-Importer/blob/master/templates/import.php)), but if this does affect things outside the importing page, a class could be added to the [`welcome-panel` div](https://github.com/humanmade/WordPress-Importer/blob/master/templates/import.php#L34) wrapper to specify that it's a `wp-import welcome-panel` and then these CSS rules could be applied to `wp-import.welcome-panel` or just `wp-import`.
@jazzsequence
Copy link
Author

With updated styles:
Screen Shot 2023-03-07 at 2 48 54 PM

Without
Screen Shot 2023-03-07 at 2 49 37 PM

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant