ionic-framework icon indicating copy to clipboard operation
ionic-framework copied to clipboard

feat: allow noOutlet in Tab

Open riderx opened this issue 2 years ago • 4 comments

Pull request checklist

Please check if your PR fulfills the following requirements:

  • [ ] Tests for the changes have been added (for bug fixes / features)
  • [ ] Docs have been reviewed and added / updated if needed (for bug fixes / features)
  • [ ] Build (npm run build) was run locally and any changes were pushed
  • [ ] Lint (npm run lint) has passed locally and any fixes were made for failures

Pull request type

Please check the type of change your PR introduces:

  • [ ] Bugfix
  • [X] Feature
  • [ ] Code style update (formatting, renaming)
  • [ ] Refactoring (no functional changes, no api changes)
  • [ ] Build related changes
  • [ ] Documentation content changes
  • [ ] Other (please describe):

What is the current behavior?

Each framework wrapper couples the ion-router-outlet within ion-tabs, prevent developers from using ion-tabs in a standalone context or passing in another router solution.

Issue URL: https://github.com/ionic-team/ionic-framework/issues/25184

What is the new behavior?

  • Allow IonTab and IonTabbar to be use without any router

Does this introduce a breaking change?

  • [ ] Yes
  • [X] No

Other information

I didn't find where in the React wrapper this is implemented so I didn't make a change, for Angular i'm not sure about what to change

riderx avatar Apr 25 '22 18:04 riderx

Thanks for taking the initial discovery at this feature!

To set up proper expectations on reviewing/merging; the team still needs to review my open feature request. Once the team agrees on the feature, we will internally create a design document that covers the necessary changes to each framework and discuss any possible issues with the suggested approach. Finally, the team would schedule the feature for a future release.

sean-perkins avatar Apr 25 '22 20:04 sean-perkins

Thanks for the process info, could be interesting to have this process more public, to share and propose things!

riderx avatar Apr 25 '22 20:04 riderx

@sean-perkins do you have a plan to include this in release 6.2 ?

or should I not wait and make my own code ?

KANekT avatar Jul 11 '22 06:07 KANekT

@KANekT this feature will not be included in the 6.2 release.

You are always able to maintain a fork of the code with your required changes in the interim.

The Ionic team will still need to complete an internal feature design document for this feature before committing it to a feature release.

sean-perkins avatar Jul 11 '22 14:07 sean-perkins

Thank you for the PR! We greatly appreciate the work you've put in, and the ability to use tabs without a router outlet is definitely something we're interested in adding to Ionic.

However, after talking this through with the team, we've decided it would be best to instead add this feature as part of an upcoming refactor we're making to the routing code that should simplify things greatly. This would allow us to add the feature to all three frameworks at once, rather than just Vue, and avoid any unnecessary extra work on anyone's part.

As such, I'm going to go ahead and close this PR out, but do let me know if you have any questions. Thank you again!

amandaejohnston avatar Nov 08 '22 18:11 amandaejohnston

thanks @amandaejohnston for the insights. where we can follow the work you doing in the routing ? if there no public access to this i would have prefer to keep it open until release happen. otherwise it make us loose the track as community members. if we keep it open or create public issue we could subscribe to the issue and know when that ready

riderx avatar Nov 09 '22 01:11 riderx

The refactoring work will be mostly internal, but the original feature request for using tabs without a router outlet will remain open: https://github.com/ionic-team/ionic-framework/issues/25184 Updates on that front will be posted there as we have more to share.

amandaejohnston avatar Nov 09 '22 15:11 amandaejohnston

thanks !

riderx avatar Nov 09 '22 15:11 riderx