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

[WIP] Add style presets #2276

Open
wants to merge 15 commits into
base: develop
Choose a base branch
from
Open

[WIP] Add style presets #2276

wants to merge 15 commits into from

Conversation

zackkatz
Copy link
Member

@zackkatz zackkatz commented Feb 5, 2025

This PR resolves #1917 and is a start to including more default style options.

This is based on the work done in Dashboard Views and uses the code structure of the work done in the Lightbox Provider functionality.

Todo:

  • Create the Orbital theme CSS based on Gravity Forms' Orbital theme
  • Remove styles that don't look as good
  • Improve the appearance of the themes in the admin (either using styles that actually reflect the designs or provide screenshots)
  • Prevent the CSS files generated during build being included in plugin bundle

💾 Build file (8b4b977).

zackkatz added 15 commits June 17, 2024 00:38
This needs a lot of work, but I wanted to get a starting point going.

Here's a video showing that it works to load the View styles, it also applies the styles in the block editor, and starts to look okay.

- [ ] We need to compile the CSS from source
- [ ] Add the GF Orbital theme CSS
- [ ] Figure out how to handle namespacing per-View. Should we load the CSS into memory and then output inline? We discussed this option at WCEU 2024.
- [ ] Clean up copypasta
- [ ] Use a nicer dropdown that can show a preview of the styles
- [ ] …Probably many other things.

https://www.loom.com/share/ae7f0b460b5f43418dbf98a9faec3db6
This is a placeholder for now.
This is based on the work for the Dashboard Views.
This is very work-in-progress, still trying to figure out the best way to show the different frameworks.

https://i.gravitykit.com/i/oGyXVm
We only need to instantiate the styles when getting their name, and only then because the names need to be translated.
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.

Create a Styles tab in the View Settings
1 participant