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

Add Elements theme #559

Merged
merged 16 commits into from
Nov 15, 2022
Merged

Add Elements theme #559

merged 16 commits into from
Nov 15, 2022

Conversation

shalvah
Copy link
Contributor

@shalvah shalvah commented Nov 15, 2022

Adds a new theme (finally 🎉 ). The theme is based on the Stoplight Elements look, so is named elements. Of course, it isn't exactly the same. There are stylistic differences, some things are removed to reduce complexity, and ours is a scrollable single page.

image

image

Also, this work led to a few improvements on the default theme, and I'll be going back to add even more soon

There's probably still a lot missing. Officially, this is still "beta". At the very least, I need to refactor and extract the duplicated code between both themes' Try It Out implementation to a common base.

  • I had to "reinvent" some of the Scribe components, because they were written with a specific theme in mind. In the next major, we'll probably move these into the respective theme folders.

  • The links property of $metadata sent to the view was also deprecated and will be removed in the next major.

@shalvah shalvah merged commit 5752f44 into master Nov 15, 2022
@shalvah shalvah deleted the elements-theme branch November 15, 2022 21:07
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