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

[iPad support] Custom tab navigator for native tablet, with side nav #3465

Open
wants to merge 10 commits into
base: main
Choose a base branch
from

Conversation

mozzius
Copy link
Member

@mozzius mozzius commented Apr 10, 2024

Stacked on #1569

This PR create a custom navigatior for the native tablet layout, mimicking the web tablet layout. However, the web navigator is in a flat structure, whereas on native it needs to maintain a stack navigator for each tab.

Simulator Screenshot - iPad Pro (11-inch) (4th generation) - 2024-04-03 at 13 46 18

Test plan

  1. Verify that the left nav is working for all tabs
  2. Make sure no links are broken, especially to the pages that have been turned into tabs
  3. Check that the header doesn't overlap the feed
  4. Check that there's no emoji picker in the composer, and other random polish things like that

@mozzius mozzius force-pushed the samuel/ipad-fixes-2 branch from 842f0c9 to b020554 Compare April 10, 2024 14:13
Copy link

github-actions bot commented Apr 10, 2024

Old size New size Diff
8.01 MB 8.01 MB 320 B (0.00%)

@mozzius mozzius changed the base branch from ipad-fixes to main April 10, 2024 14:29
@mozzius mozzius mentioned this pull request Apr 10, 2024
@mozzius mozzius force-pushed the samuel/ipad-fixes-2 branch 2 times, most recently from 6a7ebff to e81bb86 Compare April 10, 2024 15:08
@mozzius mozzius marked this pull request as ready for review April 10, 2024 15:08
@mozzius mozzius force-pushed the samuel/ipad-fixes-2 branch from e81bb86 to da1b956 Compare June 10, 2024 21:17
Copy link

render bot commented Jun 10, 2024

@mozzius mozzius force-pushed the samuel/ipad-fixes-2 branch 2 times, most recently from 30915b2 to 7971a3e Compare June 10, 2024 22:12
pfrazee and others added 6 commits November 9, 2024 10:26
…3385)

* fix fab position

* show left nav

* custom tab navigator for tablets

* re-disable right nav on tablet

* fix unread count not being rounded

* use mobile subpage header on native tablet

* fix feed being hidden under header

* rename misleading variable name

* remove drawer on native tablet, as on web tablet

* fix prompt styles on tablet

* import isWeb (ts-ignore 😡)

* disable emoji picker on tablet

* prettier format
@mozzius mozzius force-pushed the samuel/ipad-fixes-2 branch from 7971a3e to 8fcd649 Compare November 9, 2024 11:01
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