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

lateral motion #123

Merged
merged 13 commits into from
Jun 12, 2023
Merged

lateral motion #123

merged 13 commits into from
Jun 12, 2023

Conversation

Rich-Harris
Copy link
Member

This ensures there's only one direction of movement at any given moment, by abusing clip-path and having a fake background

I probably made a bit of a mess while working on this, so I might tidy some stuff up before marking it ready

@changeset-bot
Copy link

changeset-bot bot commented Jun 9, 2023

⚠️ No Changeset found

Latest commit: 18cff10

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

@Rich-Harris Rich-Harris marked this pull request as ready for review June 9, 2023 22:00
$current_menu_view = $page_selected;
});

afterNavigate(({ to }) => {
Copy link
Collaborator

Choose a reason for hiding this comment

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

This is much better! I keep forgetting Kit has these 😅

This was referenced Jun 12, 2023
@PuruVJ PuruVJ merged commit b80cf21 into feat/one-menu-to-rule-em-all Jun 12, 2023
@PuruVJ PuruVJ deleted the lateral-motion branch June 12, 2023 16:00
PuruVJ added a commit that referenced this pull request Jun 12, 2023
* Push

* Direclt render list data ratehr than component

* Fix bugs

* Imports

* Add motion

* Slightly smoother menu

* Motion aware, remove motion toggle

* Minor change

* Add external-link icon

* lateral motion (#123)

* lateral motion

* prevent flickering contextual nav

* tidy some stuff up

* reorder things

* slide down feels more natural when navigating directly

* fix links

* use navigation lifecycle

* remove unused action

* gah typescript

* Use fade out for exit

* Original namr

* Slightly slower menu close

---------

Co-authored-by: Rich Harris <git@rich-harris.dev>
Co-authored-by: Puru Vijay <devpuruvj@gmail.com>

* Incorporate changes from #120

---------

Co-authored-by: Rich Harris <hello@rich-harris.dev>
Co-authored-by: Rich Harris <git@rich-harris.dev>
PuruVJ added a commit that referenced this pull request Jun 12, 2023
* Push

* New DocsMobileNav

* Support 4.0.0

* Use JS + Accessibility

* Nav title

* Minor fixes

* Redo many things, new docsmobile nav mechanics

* Remove unused imports

* Minor fixes here and there

* Fix blockquotes

* use $page.data.nav_title

* SSR ready

* Some restructuring

* Bottom nav first concept(incomplete)

* tweak svelte recent searches

* Better tap highlight color

* Tweak DocsMobileNav

* Initially hidden nav functionality

* Unified navbar

* Simplify into one single nav

* Trickify

* Use quintOut for navbar autohide

* Minor fixes

* Minor tweaks

* Nav stays under overlay when searching

* Pad the button

* color mixed sk-theme-1

* Add changeset

* Update changeset

* Change icons a bit

* One menu to rule em all (#118)

* Push

* Direclt render list data ratehr than component

* Fix bugs

* Imports

* Add motion

* Slightly smoother menu

* Motion aware, remove motion toggle

* Minor change

* Add external-link icon

* lateral motion (#123)

* lateral motion

* prevent flickering contextual nav

* tidy some stuff up

* reorder things

* slide down feels more natural when navigating directly

* fix links

* use navigation lifecycle

* remove unused action

* gah typescript

* Use fade out for exit

* Original namr

* Slightly slower menu close

---------

Co-authored-by: Rich Harris <git@rich-harris.dev>
Co-authored-by: Puru Vijay <devpuruvj@gmail.com>

* Incorporate changes from #120

---------

Co-authored-by: Rich Harris <hello@rich-harris.dev>
Co-authored-by: Rich Harris <git@rich-harris.dev>

---------

Co-authored-by: Rich Harris <hello@rich-harris.dev>
Co-authored-by: Rich Harris <git@rich-harris.dev>
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.

2 participants