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

feat(website): migrate to nextra #12947

Merged
merged 12 commits into from
Jan 17, 2023
Merged

feat(website): migrate to nextra #12947

merged 12 commits into from
Jan 17, 2023

Conversation

dionysuzx
Copy link
Collaborator

@dionysuzx dionysuzx commented Jan 14, 2023

You can view the deployment here: https://website-git-feat-nextra-website-taikoxyz.vercel.app/

summary of changes

  • update from docusaurus v2 -> nextjs 13 (nextra)
  • upgrades from mdx v1 -> mdx v2
  • removes docusaurus css pollution, now it is using tailwind
  • darken the overall theme
  • shiki code highlighting
  • add full (in memory) text search for docs
  • add taiko status page to navigation bar
  • update bridge links to point to new faucet link
  • lighthouse performance bump, much faster

the bad parts

  • no more twitter/reddit in the navigation bar, for now. (can fix this later, but it is in footer). (eventually we may want to separate out "landing page" and "docs page" which sit at taiko.xyz and docs.taiko.xyz, but can do this effort after flushing out the docs with whitepaper material).
  • should optimize the SEO a bit

@codecov
Copy link

codecov bot commented Jan 14, 2023

Codecov Report

Merging #12947 (f684736) into main (59276fe) will decrease coverage by 0.26%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main   #12947      +/-   ##
==========================================
- Coverage   66.67%   66.41%   -0.27%     
==========================================
  Files         113      113              
  Lines        3034     3034              
  Branches      356      356              
==========================================
- Hits         2023     2015       -8     
- Misses        935      943       +8     
  Partials       76       76              
Flag Coverage Δ *Carryforward flag
bridge-ui 95.05% <ø> (ø)
protocol 59.10% <ø> (-0.51%) ⬇️
relayer 69.10% <ø> (ø) Carriedforward from d50f5da
ui 100.00% <ø> (ø) Carriedforward from d50f5da

*This pull request uses carry forward flags. Click here to find out more.

Impacted Files Coverage Δ
packages/protocol/contracts/bridge/Bridge.sol 59.37% <ø> (ø)
...ges/protocol/contracts/libs/LibAnchorSignature.sol 89.47% <0.00%> (-10.53%) ⬇️
...es/protocol/contracts/thirdparty/LibMerkleTrie.sol 83.13% <0.00%> (-7.23%) ⬇️

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@dong77
Copy link
Contributor

dong77 commented Jan 15, 2023

Looks great, the navbar font size is a bit too small (even bigger than the footer fonts)

dantaik
dantaik previously approved these changes Jan 15, 2023
@dionysuzx dionysuzx enabled auto-merge (squash) January 15, 2023 21:53
dantaik
dantaik previously approved these changes Jan 16, 2023
@dantaik dantaik changed the title feat: migrate to nextra feat(website): migrate to nextra Jan 16, 2023
@dionysuzx dionysuzx merged commit ac11959 into main Jan 17, 2023
@dionysuzx dionysuzx deleted the feat-nextra-website branch January 17, 2023 01:22
@github-actions github-actions bot mentioned this pull request Jan 18, 2023
davidtaikocha pushed a commit that referenced this pull request Jan 18, 2023
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.

4 participants