[GitHub] cordova-plugin-inappbrowser pull request #210: Ios style improvements

Wed, 18 Jan 2017 18:32:06 -0800 

GitHub user reidab opened a pull request:

    https://github.com/apache/cordova-plugin-inappbrowser/pull/210

    Ios style improvements

    <!--
    Please make sure the checklist boxes are all checked before submitting the 
PR. The checklist
    is intended as a quick reference, for complete details please see our 
Contributor Guidelines:
    
    http://cordova.apache.org/contribute/contribute_guidelines.html
    
    Thanks!
    -->
    
    ### Platforms affected
    
    iOS
    
    ### What does this PR do?
    
    I've been working on a branch that brings several visual and configuration 
improvements to the iOS InAppBrowser view, along with a little refactoring of 
the view setup code.
    
    This PR:
    * breaks up the large `CDVInAppBrowserViewController createView` method 
into several smaller helper methods to handle the creation of individual 
subviews
    * gives the `addressLabel` a wrapping view to provide flexibility of 
background color / padding
    * changes the default toolbar style from `UIBarStyleBlackOpaque` to 
`UIBarStyleDefault` to match the system default
    * adds a `toolbarstyle` configuration option that accepts `default` or 
`black`
    * sets the status bar style and background color to match the toolbarstyle, 
so that top toolbars can blend nicely into the status bar
    * replaces the unicode navigation arrows (◄►) with drawn arrows that 
match those used in Safari
    * adds a `tintcolor` configuration option that accepts hex strings like 
`f8a027`. If not provided, or set to `default`, controls will be rendered in 
the default iOS blue, or in white when `toolbarstyle=black` is set.
    
    ### What testing has been done on this change?
    
    Simulator and device iOS 10. Ran plugin auto tests locally.
    
    ### Checklist
    - [x] [Reported an issue](http://cordova.apache.org/contribute/issues.html) 
in the JIRA database
    - [x] Commit message follows the format: "CB-3232: (android) Fix bug with 
resolving file paths", where CB-xxxx is the JIRA ID & "android" is the platform 
affected.
    - [ ] Added automated test coverage as appropriate for this change.
    
    ### Screenshots!
    
    #### Old Default
    
    <img 
src="https://cloud.githubusercontent.com/assets/13192/22090934/093ac000-dda9-11e6-8e95-09ca9929fa68.png";
 width=250>
    
    ---
    
    #### New Default 
    
    <img 
src="https://cloud.githubusercontent.com/assets/13192/22091065/b22beb12-dda9-11e6-8d6c-29b2b5715869.png";
 width=250>
    
    ---
    
    #### `location=no`
    
    <img 
src="https://cloud.githubusercontent.com/assets/13192/22091092/ef0b84f2-dda9-11e6-82e8-405aa7ccde9a.png";
 width=250>
    
    ---
    
    #### `location=no,toolbarposition=top`
    
    <img 
src="https://cloud.githubusercontent.com/assets/13192/22091124/1bcdd7d8-ddaa-11e6-860b-873c486e627a.png";
 width=250>
    
    ---
    
    #### `location=no,toolbarposition=top,tintcolor=F8A027`
    
    <img 
src="https://cloud.githubusercontent.com/assets/13192/22091138/32e9555a-ddaa-11e6-9801-8dd57863f2eb.png";
 width=250>
    
    ---
    
    #### `location=no,toolbarposition=top,tintcolor=F8A027,toolbarstyle=black`
    
    <img 
src="https://cloud.githubusercontent.com/assets/13192/22091158/4f9a646e-ddaa-11e6-9201-74f672a1611e.png";
 width=250>
    
    ---
    
    #### `location=no,toolbarposition=top,toolbarstyle=black`
    
    <img 
src="https://cloud.githubusercontent.com/assets/13192/22091192/8726d4b2-ddaa-11e6-8462-63f300d840e0.png";
 width=250>
    
    ---
    
    #### `location=no,toolbarstyle=black`
    
    <img 
src="https://cloud.githubusercontent.com/assets/13192/22091204/9ce2720c-ddaa-11e6-9799-bd57497cfbeb.png";
 width=250>
    
    ---

You can merge this pull request into a Git repository by running:

    $ git pull https://github.com/reidab/cordova-plugin-inappbrowser 
ios-style-improvements

Alternatively you can review and apply these changes as the patch at:

    https://github.com/apache/cordova-plugin-inappbrowser/pull/210.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

    This closes #210
    
----
commit 358c052b1d699f4115e1276d116265171532152d
Author: Reid Beels <[email protected]>
Date:   2017-01-18T05:58:45Z

    CB-12367: (ios) Extract UIWebView setup into buildWebView

commit 19ea6d117fce6d20c4c4bcb1f00169f3db96c751
Author: Reid Beels <[email protected]>
Date:   2017-01-18T06:00:27Z

    CB-12367: (ios) Extract UIActivityIndicatorView setup into buildSpinner

commit 362df5f9764b1073cdc7ffe6579ae378ae4048d3
Author: Reid Beels <[email protected]>
Date:   2017-01-18T06:02:57Z

    CB-12367: (ios) Extract UIToolbar setup into buildToolbar

commit 5d97ab2fbea2c51685db6230953c65d0d746dbf0
Author: Reid Beels <[email protected]>
Date:   2017-01-18T06:05:00Z

    CB-12367: (ios) Extract address UILabel setup into buildAddressLabel

commit ff05f6e1a83a0205851ca776fed990812cc6db15
Author: Reid Beels <[email protected]>
Date:   2017-01-18T06:06:59Z

    CB-12367: (ios) Extract toolbar button setup into separate methods

commit e6c906cdb75a673b25d24b7036678be891e7d78d
Author: Reid Beels <[email protected]>
Date:   2017-01-18T06:27:58Z

    CB-12367: (ios) Allow setting the UIBarStyle of the toolbar to `black` or 
`default`

commit c909fc75e5ecadbaca24697d54ab08acca37acef
Author: Reid Beels <[email protected]>
Date:   2017-01-18T08:02:30Z

    CB-12367: (ios) Replace arrow strings with iOS 7 style back/forward arrows

commit 63cdd716a5fca5bf2cc20f3b40af6d2e4cdc93b7
Author: Reid Beels <[email protected]>
Date:   2017-01-18T09:25:37Z

    CB-12367: (ios) Set the status bar background style to match the toolbar 
style

commit e86bf9d16017c55891e8f63f426d045682186a30
Author: Reid Beels <[email protected]>
Date:   2017-01-18T09:26:01Z

    CB-12367: (ios) Set the status bar style to match the toolbar style

commit 744ebdbec58795eb9675fd12efe1e0a5189aaf71
Author: Reid Beels <[email protected]>
Date:   2017-01-18T18:07:04Z

    CB-12367: (ios) Set toolbar tint color to white if toolbar is dark

commit 068160d892f81d7ee371a70c2c9c19a8b2a500f9
Author: Reid Beels <[email protected]>
Date:   2017-01-18T23:35:06Z

    CB-12367: (ios) Don't subtract toolbar height from webview height
    
    We want the toolbar to overlap for a glassy blur

commit 0ffa32a779a0ed5a2ba18f3b170752339f01e0c9
Author: Reid Beels <[email protected]>
Date:   2017-01-19T00:14:20Z

    CB-12367: (ios) Wrap addressLabel in a UIView to handle inset/background

commit 904ca968cdb37187fc3e998c2ef0566990b448f4
Author: Reid Beels <[email protected]>
Date:   2017-01-19T01:12:24Z

    CB-12367: (ios) Allow setting the view's tint color via `tintcolor=00ff00`

----


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at [email protected] or file a JIRA ticket
with INFRA.
---

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to