Automattic / WP-Job-Manager

Manage job listings from the WordPress admin panel, and allow users to post jobs directly to your site.
https://wpjobmanager.com
GNU General Public License v3.0
893 stars 366 forks source link

Use theme colors for job overlay, improve dashboard on small screens #2810

Closed yscik closed 2 months ago

yscik commented 2 months ago

Changes Proposed in this Pull Request

Testing Instructions

New or Updated Hooks and Templates

Screenshot / Video

image image
Plugin build for df2ed4356240699fa85e622a7ebadc2c53d45716
📦 Download plugin zip
▶️ Open in playground
yscik commented 2 months ago

So regarding CSS variables, the new styles rely pretty heavily on it, and it makes it pretty easy to customize things with just a few variables. This is an alternative to rewriting or overriding everything with CSS, which is what they currently have to do. I'm putting together a documentation page with some examples to get started.

This approach also makes it easy to add an UI on top in the future, where they can change the colors in the site editor or the setup wizard.

This PR adds the color detections so styles are not broken out of the box on various themes. Most of it is for the new overlay modal, where we'd end up with stuff like white links on white background if it's a dark theme.

yscik commented 2 months ago

Thanks for the review @gikaragia, all should be fixed now!