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-15424] iOS: adds support for pageIndicatorTintColor and currentPageInd... #6781

Closed
wants to merge 3 commits into from

Conversation

manumaticx
Copy link
Contributor

...icatorTintColor to ScrollableView

The credits for this code goes entirely to Samuele Coppede: https://github.com/boardmain/TitaniumMods

…IndicatorTintColor to ScrollableView

The credits for this code goes entirely to Samuele Coppede: https://github.com/boardmain/TitaniumMods
@manumaticx manumaticx changed the title [TIMOB-15424] adds support for pageIndicatorTintColor and currentPageInd... [TIMOB-15424] iOS: adds support for pageIndicatorTintColor and currentPageInd... Apr 15, 2015
@hansemannn
Copy link
Collaborator

Hi @manumaticx, we'd welcome to get this merged. Can you please do the following things before:

  • Resolve the merge conflicts with the latest master
  • Ensure that the (new) default values for the colors don't break existing app designs.
  • Create a JIRA-ticket here

Thanks!

@manumaticx
Copy link
Contributor Author

@hansemannn isn't this existing ticket the right one?: https://jira.appcelerator.org/browse/TIMOB-15424

@hansemannn
Copy link
Collaborator

Oh sure, didn't see in the first place. The docs need the since: "6.0.0" in it and for some reason travis CI failed a few months ago. Will retrigger it after the doc updates. Did you ensure that the (new) default values for the colors don't break existing app designs?

@manumaticx
Copy link
Contributor Author

Not yet. I'll update this PR asap after testing default colors.

Manuel Lehner added 2 commits January 28, 2016 14:39
@manumaticx
Copy link
Contributor Author

@hansemannn That color issue was indeed a good pointer because gray is not the default color but a semi-transparent white (which looks gray on the default black background).
simulator screen shot 28 01 2016 14 27 03
(for reference: this is how the paging control should look like on blue background)

I've updated this PR with resolved conflicts and added the since property to the docs.

@hansemannn
Copy link
Collaborator

I thought more of nil'ing the value so iOS uses its own default values. Especially instead of creating an alpha channel manually.

@manumaticx
Copy link
Contributor Author

Okay, I have to admit that I'm neither an iOS developer nor the author of that fix. I just took Samuels patch and created the PR since nobody else did. Because this solutions has worked for me since Titanium 3.0.2.GA and many Ti users used it too.

However, we might should just close this, hoping @boardmain comes up with his own pull request ;)

@hansemannn
Copy link
Collaborator

All good! I'll take your PR and modify it in another commit to credit your work. Thank you!

@manumaticx
Copy link
Contributor Author

I was hoping you'd say that! :) (I'm busy with other stuff atm) - Thanks! (and don't miss: credits goes to Samuele Coppede)

@hansemannn
Copy link
Collaborator

Hey there! Finally found the time to finish the PR (#7888). I did the following changes:

  • Rename pageIndicatorTintColor to pageIndicatorColor
  • Rename currentPageIndicatorTintColor to currentPageIndicatorColor
  • Change default values to nil (which makes iOS to use the system-default - futureproof)
  • Change the since key from 6.0.0 to 5.4.0

That's it. Thank you!

@hansemannn hansemannn closed this Mar 27, 2016
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

2 participants