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

chore: update demo #799

Merged
merged 12 commits into from
Feb 8, 2024
Merged

chore: update demo #799

merged 12 commits into from
Feb 8, 2024

Conversation

hgray-instawork
Copy link
Collaborator

@hgray-instawork hgray-instawork commented Feb 2, 2024

Updating the demo to include example using <navigator> HXML source.

  • Created a toggle in the header to switch between external navigator (legacy) and internally generated navigator using new <navigator> syntax
  • Fixed an issue with the 'changed' file containing invalid attributes (can't have scrolling and safe enabled on the same view)
  • Added 'back' example for behavior triggers (this will only work using the internal navigator)

Examples showing different functionality with 'legacy' and 'updated' navigation:

Close action Back trigger
close back

Asana: https://app.asana.com/0/1204008699308084/1206290261007827/f

Copy link
Collaborator

@flochtililoch flochtililoch left a comment

Choose a reason for hiding this comment

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

I think we should revise the approach to:

  • separate out the code for internal and external navigator - most users will use one or the other, and having them both on the same file will make it harder to reason about.
  • handle the toggling via a custom behavior - this will allow for the toggle UI to be rendered within Hyperview itself and also serve as a showcase for custom HV behaviors.

@hgray-instawork hgray-instawork changed the base branch from master to hardin/prettier-demo February 7, 2024 22:21
Base automatically changed from hardin/prettier-demo to master February 7, 2024 23:05
# Conflicts:
#	demo/src/HyperviewScreen.tsx
#	demo/src/Navigator.tsx
demo/App.tsx Outdated Show resolved Hide resolved
demo/App.tsx Outdated Show resolved Hide resolved
demo/App.tsx Outdated Show resolved Hide resolved
demo/App.tsx Outdated Show resolved Hide resolved
demo/App.tsx Show resolved Hide resolved
demo/App.tsx Outdated Show resolved Hide resolved
demo/App.tsx Outdated Show resolved Hide resolved
demo/App.tsx Outdated Show resolved Hide resolved
demo/App.tsx Outdated Show resolved Hide resolved
demo/App.tsx Outdated Show resolved Hide resolved
examples/_includes/base.xml.njk Outdated Show resolved Hide resolved
examples/index_hxml.xml.njk Outdated Show resolved Hide resolved
examples/index_navigator.xml.njk Outdated Show resolved Hide resolved
Copy link
Collaborator

@flochtililoch flochtililoch left a comment

Choose a reason for hiding this comment

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

LGTM

@hgray-instawork hgray-instawork merged commit 50f3b8f into master Feb 8, 2024
5 checks passed
@hgray-instawork hgray-instawork deleted the hardin/demo-update branch February 8, 2024 17:06
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