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

[TIMOB-18914] Android: Mark splitActionBar as removed #9156

Merged
merged 6 commits into from Aug 28, 2017

Conversation

ypbnv
Copy link
Contributor

@ypbnv ypbnv commented Jun 19, 2017

Remove from source. Mark in documentation as "removed".

JIRA: https://jira.appcelerator.org/browse/TIMOB-18914

Remove from source. Mark in documentation as "removed".
@ypbnv ypbnv added this to the 6.2.0 milestone Jun 19, 2017
@ypbnv ypbnv requested a review from garymathews June 19, 2017 15:11
@ypbnv
Copy link
Contributor Author

ypbnv commented Jun 20, 2017

@hansemannn Reverted the code removal. Added only deprecation tag and some notes. I guess we can link the Ti.UI.Toolbar in the notes once we have it moved on both platforms in the proper module.

@lokeshchdhry
Copy link
Contributor

@ypbnv , can QE do a FR for this ? Is CR pending ?

@@ -784,6 +784,9 @@ properties:
type: Boolean
since: 3.6.0
availability: creation
deprecated:
since: 6.2.0
notes: Deprecated in AppCompat theme. The same behaviour can be achived by using Toolbar.
Copy link
Collaborator

Choose a reason for hiding this comment

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

So it still work? When you deprecate it, it still has to work. In case it does not anymore, it has to go into a major version (7.0.0).

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Yes, it should work. On Titanium SDK less than 4.0.0. Material Theme was added in that version and it does not support this property.

@hansemannn hansemannn modified the milestones: 7.0.0, 6.2.0 Aug 15, 2017
Copy link
Contributor

@garymathews garymathews left a comment

Choose a reason for hiding this comment

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

CR: PASS

@ypbnv needs backporting to 6_2_X

@mukherjee2
Copy link
Contributor

Passes FR.

@mukherjee2 mukherjee2 merged commit 07e0880 into tidev:master Aug 28, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants