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

perf(docs): improve rendering performance of example's pages #3549

Merged
merged 14 commits into from
Apr 22, 2019

Conversation

layershifter
Copy link
Member

@layershifter layershifter commented Apr 3, 2019

This PR is focused on improving docs usage experience.

Extra responsive pages

Performance is gained by using placeholders for invisible components.
Each page is loading extra fast now 🚀

Simplify component controls

I removed Show HTML control as it was done in Stardust by two reasons:

  • it's more usable
  • it simplifies code very much

Sidebar is fixed 🔨

Docs sidebar was moved to a higher level and now it will be not remounted on page switch.

…React into improve/perf

# Conflicts:
#	docs/src/components/ComponentDoc/ComponentDoc.js
…React into improve/perf

# Conflicts:
#	docs/src/components/ComponentDoc/ComponentControls/ComponentControls.js
#	docs/src/components/ComponentDoc/ComponentControls/ComponentControlsCodeSandbox.js
#	docs/src/components/ComponentDoc/ComponentExample/ComponentExampleRenderEditor.js
#	docs/src/components/ComponentDoc/ComponentExample/ComponentExampleRenderSource.js
#	docs/src/components/ComponentDoc/ComponentProp/ComponentPropEnum.js
#	docs/src/components/ComponentDoc/ComponentProp/ComponentPropEnumValue.js
#	docs/src/components/ComponentDoc/ComponentProp/ComponentPropFunctionSignature.js
#	docs/src/components/ComponentDoc/ComponentProp/ComponentPropHeader.js
#	docs/src/components/ComponentDoc/ComponentProps/ComponentPropsComponents.js
#	docs/src/components/ComponentDoc/ComponentSidebar/ComponentSidebar.js
#	docs/src/components/ComponentDoc/ComponentTable/ComponentTableHeader.js
…React into improve/perf

# Conflicts:
#	docs/src/components/ComponentDoc/ComponentProps/ComponentProps.js
#	docs/src/components/ComponentDoc/ComponentSidebar/ComponentSidebarSection.js
…React into improve/perf

# Conflicts:
#	docs/src/components/ComponentDoc/ComponentProps/ComponentProps.js
#	docs/src/components/ComponentDoc/ComponentSidebar/ComponentSidebarSection.js
@codecov-io
Copy link

codecov-io commented Apr 10, 2019

Codecov Report

Merging #3549 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master    #3549   +/-   ##
=======================================
  Coverage   99.81%   99.81%           
=======================================
  Files         174      174           
  Lines        2730     2730           
=======================================
  Hits         2725     2725           
  Misses          5        5

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update ffa3874...4469ef1. Read the comment docs.

…React into improve/perf

# Conflicts:
#	docs/src/components/CodeEditor/CodeEditor.js
#	docs/src/components/CodeSnippet/CodeSnippet.js
#	docs/src/pages/Usage.mdx
@layershifter layershifter merged commit 3f0c8ea into master Apr 22, 2019
@delete-merged-branch delete-merged-branch bot deleted the improve/perf branch April 22, 2019 10:30
mbakiev pushed a commit to mbakiev/Semantic-UI-React that referenced this pull request Jun 17, 2019
…c-Org#3549)

* wip

* Merge branch 'master' of https://github.com/Semantic-Org/Semantic-UI-React into improve/perf

# Conflicts:
#	docs/src/components/ComponentDoc/ComponentProps/ComponentProps.js
#	docs/src/components/ComponentDoc/ComponentSidebar/ComponentSidebarSection.js

* restore changes

* clean up

* restore changes

* clean up

* optimizations

* add note

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

Successfully merging this pull request may close these issues.

2 participants