-
Notifications
You must be signed in to change notification settings - Fork 4.6k
Reorganize the site editor to introduce Browse Mode. #44770
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
Merged
Merged
Changes from 1 commit
Commits
Show all changes
84 commits
Select commit
Hold shift + click to select a range
2090043
Bootstrap the new admin navigation screen (start with the site editor)
youknowriad 16b18f8
Fix unit tests
youknowriad 82d3778
Tweak the sidebar animation
youknowriad c72f552
Fix toggle button height
youknowriad 7b43bc2
Improve the animation
youknowriad 525ce71
Add save button to the canvas header
youknowriad 6ae91c8
Fix e2e tests
youknowriad 11ba051
Auto switch to edit mode when clicking templates or template parts
youknowriad 87df95d
Adjust menu spacing and animation vfx
jameskoster c08add2
Small style refactoring
youknowriad d6983d3
Add the currently edited template title to the view mode header
youknowriad 1d81206
Add a home button
youknowriad 2269126
a11y improvements
youknowriad 54b547f
Fix more e2e tests
youknowriad c5cf7d9
Fix performance tests
youknowriad b5cae0e
more e2e test fixes
youknowriad 5dcaea3
smoother animation
youknowriad 096b23b
Fix the site editor toggle
youknowriad 62328a6
Remove duplicated doc
youknowriad 86914ed
Fix the site editor on mobile
youknowriad ab14e6f
Remove the editor menu item
youknowriad c070c55
Fix header padding
youknowriad 0e139af
Selecting a template doesn't move into edit mode right away
youknowriad 7331a5e
Redesign the sidebar title
youknowriad 1e1b378
Remove home and save buttons
youknowriad ce5f07f
Scale down the frame content
youknowriad 138bb5b
Better spacing and colors for the menu items
youknowriad da7fd8d
Tweak the sidebar width
youknowriad 0a04fe4
Remove home page template setting
youknowriad b964e7c
Move the edit button outside of the canvas
youknowriad fd12005
only apply radius in view mode
youknowriad 1505ab7
Fix sidebar alignment
youknowriad dd9a0f6
Remove the background and fix radius
youknowriad adbdafc
Update spacing
jameskoster eddd2b5
Increase frame border radius
jameskoster 3cd8109
Remove iframe background and add elevation
jameskoster b2e08f7
Fix list header appearance
jameskoster b2aff8a
Use -size for nav margin
jameskoster 5594734
Consistent canvas style
youknowriad 133bdde
Remove back button
youknowriad 06d7833
Fix template parts in view mode
youknowriad 63a4180
Reshuffle the structure to animate the header from the top
youknowriad cd51ae8
Keep the sidebar visible while animating the canvas
youknowriad 913322f
Scaling the whole canvas
youknowriad 64a75b7
Remove useless code
youknowriad e78bd97
Remove the sidebar animation
youknowriad 9b437e1
Restore the sidebar animation
youknowriad d01e26a
Fix text color
youknowriad 1a0f80e
Animate the iframe
youknowriad 68fd4af
Move the edit button to the sidebar
youknowriad e954612
Fix mobile
youknowriad a4fbe48
Improve the canvas animation
youknowriad 0b36c3c
Remove scaling and simplify animation
youknowriad 258622b
Tweak animation easing and frame shadow.
mtias 4b312eb
Hide the sidebar behind the canvas
youknowriad 8d7e1d5
Improve the animation
youknowriad 72f74fc
Restore the sidebar padding
youknowriad c228eb6
Rebase cleanup
youknowriad 0263436
Update shadow on the frame.
mtias 7b92eb8
Fix inserter height
youknowriad 3aaad16
Improve the animation of the edit button
youknowriad e8918cd
Move the edit button next to design
youknowriad 940250e
Fix e2e tests
youknowriad ae2c7d0
fix more e2e tests
youknowriad 82af6ca
Restore aria region
youknowriad fb76172
Fix the padding of the template list
youknowriad 3efae5a
Hide edit button in template list
youknowriad 39b86fd
Remove link from the site title
youknowriad 16de8a2
Fix post rebase
youknowriad d5bc732
Quit zoom-out when switching to view mode
youknowriad 96e43b3
Switch to edit mode when creating templates
youknowriad 7f91013
Remove useless condition
youknowriad c27764a
Code simplification
youknowriad d54f42b
Hide publish flow in view mode
youknowriad 5676e75
Fix layout shift
youknowriad ecfc594
Remove useless showDropdown
youknowriad 04f8d7e
Remove useless prop
youknowriad d53a47f
Split site icon and title component
youknowriad 732cea6
Fix showIconLabels behavior
youknowriad ad0ff98
Restore the onError prop
youknowriad acb368f
Fix code editor position
youknowriad 3191104
Catch the error in setTemplate
youknowriad d5dcbc7
Remove list view styling changes
youknowriad a8683bd
Allow scrolling in view mode
youknowriad File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Remove iframe background and add elevation
- Loading branch information
commit 3cd810903927682116e0d5409aba4538e95a4b2a
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Don't you think we should add a fallback background for the pages that may not set any (list pages...)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This was causing a visual artefact in browse mode that was important to eliminate. I added a new style in 20784db to handle list views.