View Issue Details
| ID | Project | Category | View Status | Date Submitted | Last Update |
|---|---|---|---|---|---|
| 0010769 | MMW v4 | Other | public | 2013-04-24 18:23 | 2014-01-17 15:06 |
| Reporter | rusty | Assigned To | |||
| Priority | immediate | Severity | feature | Reproducibility | always |
| Status | resolved | Resolution | fixed | ||
| Product Version | 4.1 | ||||
| Target Version | 4.1 | Fixed in Version | 4.1 | ||
| Summary | 0010769: Replace Glided skin with MetroMonkey | ||||
| Description | We should switch the Glided skin with the new MetroMonkey slim skin. Before we make the switch, we need an update to the skin that resolves performance issues (Currently, some skin redraws are so slow that they're visible. You can see this in the metro monkey skin by selecting a device, and then switching between tabs in the device configuration screen). | ||||
| Tags | No tags attached. | ||||
| Fixed in build | 1688 | ||||
|
|
Fixed in 1635 |
|
|
Leaving open till performance issues are fixed |
|
|
Raised to 'immediate' for inclusion in the next build, since performance issues have been resolved. |
|
|
verified 1636 |
|
|
We should include version 1.4.4 of the skin. Note: the skin name should display as 'Metro M (1.4.4)' (instead of Metro_M_1,4,4). |
|
|
Fixed in 1639 |
|
|
Verified 1642 |
|
|
I am sorry for re-opening this issue, but I just want to say that it is not very good idea to add the skin version to its name, especially not to its file name. I have several scripts that have css styles used with ActiveX controls for several popular skins. My scripts have detection which skin is currently active by reading the PlayerSkinNameNew3 key from the [Appearance] section in the .ini file. As you know, that key has stored the skin's file name. Because of that my .css files are named after the skins filenames, e.g. if my script finds PlayerSkinNameNew3=Glided.msz in .ini file then it will load my Glided.css file. Currently, I have added Metro M (1.4.4).css file for your Metro skin, but if you decide to update that skin and if you modify the skin's file name because of the changed version, then my scripts will not display ActiveX controls correctly with that skin anymore. |
|
|
I agree to add skin without version#. |
|
|
I've removed version number from name. |
|
|
There is a serious regression related to renaming of the skin, build 1686 starts without skin and trying to open device config throws AV because of the skin issue. Confirmed myself and can be seen in several ELFs from 1686, is also reported here: http://www.mediamonkey.com/forum/viewtopic.php?f=6&t=75128&p=381772#p381748 Edit: To reproduce: just install build 1686 to separate folder |
|
|
Fixed in 1688 |
