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

Component Explorer Improvements + Speed Boost #1013

Merged
merged 6 commits into from
Jan 7, 2019

Conversation

sghoweri
Copy link
Contributor

@sghoweri sghoweri commented Jan 7, 2019

Jira

http://vjira2:8080/browse/BDS-879?filter=-1

Summary

UI cleanup + overall improvements to the new component explorer component on the boltdesignsystem.com site.

Details

These updates should help speed up the initial and subsequent component being rendered + improves the accuracy of the component re-renders (via splitting apart the React JSON Schema Form UI into a Preact-specific component and decoupling this from the rendered component via Lit-HTML...)

Funny enough, no longer rendering this component preview in an iFrame helps a TON with the overall performance of this!

How to test

  • Check out the button and navbar docs pages in Pattern Lab and verify the component explorer is rendering as expected
  • Spot check a few other component docs pages to verify the top section of the page still check out OK (even though the navbar and button are the only ones wired up to use the component explorer at the moment)

…ountered with older buggy versions of lit-html directives
…er intro section for the component explorer UI; add pre-rendered HTML to help with initial page loading experience
…component + schema form sub-component; addresses performance and re-rendering issues encountered when only using Preact for handling the overall component rendering
@sghoweri sghoweri requested a review from danielamorse January 7, 2019 17:30
@bolt-bot
Copy link
Collaborator

bolt-bot commented Jan 7, 2019

⚡ PR built on Travis and deployed a now preview here:

@sghoweri
Copy link
Contributor Author

sghoweri commented Jan 7, 2019

@bolt-bot
Copy link
Collaborator

bolt-bot commented Jan 7, 2019

⚡ PR built on Travis and deployed a now preview here:

Copy link
Collaborator

@danielamorse danielamorse left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Tested button and navbar, worked as expected and noticeably snappier 👍

As discussed, it is not rendering in IE, but same is true on master so this is not a regression. Layout issues with IE have been fixed, too.

@sghoweri sghoweri merged commit 3f93f79 into master Jan 7, 2019
@sghoweri sghoweri deleted the feature/component-explorer-updates branch January 7, 2019 21:51
@sghoweri sghoweri mentioned this pull request Jan 8, 2019
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.

3 participants