-
Notifications
You must be signed in to change notification settings - Fork 13.5k
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
Remove mode-start
and mode-end
slots from toolbar
#14172
Comments
Revert |
After some further discussion on this we decided to move forward with the following change in naming:
This table can also be found in the breaking changes document for v4 Here is a diagram to further explain why we went with this naming convention: |
Thanks for the issue! This issue is being locked to prevent comments that are not relevant to the original issue. If this is still an issue with the latest version of Ionic, please create a new issue and ensure the template is fully filled out. |
Ionic version: (check one with "x")
[ ] 1.x (For Ionic 1.x issues, please use https://github.com/ionic-team/ionic-v1)
[ ] 2.x
[ ] 3.x
[x] 4.x
Right now we have
mode-start
andmode-end
to recreate the functionality thatstart
andend
used to have.For example:
v3
v4
I'd like to remove these and instead just use
start
andend
, but first I need to verify that this functionality can still easily be added. The old functionality caused a lot of confusion and wasn't intuitive.More info here: https://github.com/ionic-team/ionic/blob/master/BREAKING.md#attributes-renamed-4
Useful comments:
<ion-buttons start>
not working inside<ion-navbar>
#12223 (comment)The text was updated successfully, but these errors were encountered: