[ 
https://issues.apache.org/jira/browse/CB-10178?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15177069#comment-15177069
 ] 

ASF GitHub Bot commented on CB-10178:
-------------------------------------

Github user nikhilkh commented on a diff in the pull request:

    https://github.com/apache/cordova-docs/pull/528#discussion_r54830785
  
    --- Diff: www/docs/en/dev/config_ref/images.md ---
    @@ -42,79 +45,107 @@ density       | *Optional* <br/> Specified icon 
density (Android Specific)
     
     The following configuration can be used to define single default icon
     which will be used for all platforms.
    -
    -        <icon src="res/icon.png" />
    -
    +```xml
    +    <icon src="res/icon.png" />
    +```
     For each platform you can also define a pixel-perfect icons set to fit
     different screen resolutions.
     
     ##Android
    -
    -         <platform name="android">
    -                  <icon src="res/android/ldpi.png" density="ldpi" />
    -                  <icon src="res/android/mdpi.png" density="mdpi" />
    -                  <icon src="res/android/hdpi.png" density="hdpi" />
    -                  <icon src="res/android/xhdpi.png" density="xhdpi" />
    -         </platform>
    +```xml
    +    <platform name="android">
    +        <!-- 
    +            ldpi    : 36x36 px
    +            mdpi    : 48x48 px
    +            hdpi    : 72x72 px
    +            xhdpi   : 96x96 px
    +            xxhdpi  : 144x144 px
    +            xxxhdpi : 192x192 px
    +        -->
    +        <icon src="res/android/ldpi.png" density="ldpi" />
    +        <icon src="res/android/mdpi.png" density="mdpi" />
    +        <icon src="res/android/hdpi.png" density="hdpi" />
    +        <icon src="res/android/xhdpi.png" density="xhdpi" />
    +        <icon src="res/android/xxhdpi.png" density="xxhdpi" />
    +        <icon src="res/android/xxxhdpi.png" density="xxxhdpi" />
    +    </platform>
    +```
    +###See Also
    +- [Android icon guide](https://www.google.com/design/spec/style/icons.html)
    +- [Android - Supporting multiple 
screens](http://developer.android.com/guide/practices/screens_support.html)
    --- End diff --
    
    Can you comment on why it is better in this case?


> Unclear windows phone documentation for icons and splash screens
> ----------------------------------------------------------------
>
>                 Key: CB-10178
>                 URL: https://issues.apache.org/jira/browse/CB-10178
>             Project: Apache Cordova
>          Issue Type: Improvement
>          Components: Docs
>    Affects Versions: 5.4.0
>         Environment: Phonegap CLI, building to Windows platform. Testing on 
> physical windows phone device and on windows phone emulator. 
>            Reporter: Rupert McKay
>            Assignee: Nikhil Khandelwal
>            Priority: Minor
>              Labels: Docs-6.x, triaged, windows
>   Original Estimate: 1h
>  Remaining Estimate: 1h
>
> https://github.com/apache/cordova-docs/blob/master/www/docs/en/dev/config_ref/images.md
> In the section 'Configuring Icons in the CLI' the windows sections neglect to 
> mention the affect adding a .scale-100 or .scale-240 extension can have to 
> how source images get mapped to destination images. 
> The examples provided will all default to mapping onto .scale-100. 
> Recommend adding examples like the following:
> {code:xml}
>     <platform name="windows">
>         <icon src="www/res/windows/SplashScreenPhone.scale-240.png" 
> width="1152" height="1920" />
>         <icon src="www/res/windows/Square30x30Logo.scale-100.png" width="30" 
> height="30" />
>         <icon src="www/res/windows/Square44x44Logo.scale-100.png" width="44" 
> height="44" />
>         <icon src="www/res/windows/Square44x44Logo.scale-240.png" width="106" 
> height="106" />
>         <icon src="www/res/windows/Square70x70Logo.scale-100.png" width="70" 
> height="70" />
>         <icon src="www/res/windows/Square71x71Logo.scale-100.png" width="71" 
> height="71" />
>         <icon src="www/res/windows/Square71x71Logo.scale-240.png" width="170" 
> height="170" />
>         <icon src="www/res/windows/Square150x150Logo.scale-100.png" 
> width="150" height="150" />
>         <icon src="www/res/windows/Square150x150Logo.scale-240.png" 
> width="360" height="360" />
>         <icon src="www/res/windows/Square310x310Logo.scale-100.png" 
> width="310" height="310" />
>         <icon src="www/res/windows/StoreLogo.scale-100.png" width="50" 
> height="50" />
>         <icon src="www/res/windows/StoreLogo.scale-240.png" width="120" 
> height="120" />
>         <icon src="www/res/windows/Wide310x150Logo.scale-240.png" width="744" 
> height="360" />
>         <icon src="www/res/windows/Wide310x150Logo.scale-100.png" width="310" 
> height="150" />
>         <splash src="www/res/windows/SplashScreen.scale-100.png" width="620" 
> height="300" />
>     </platform>
> {code}



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscr...@cordova.apache.org
For additional commands, e-mail: issues-h...@cordova.apache.org

Reply via email to