Project:
| View Issue Details[ Jump to Notes ] | [ Issue History ] [ Print ] | |||||||
| ID | ||||||||
| 0036708 | ||||||||
| Type | Category | Severity | Reproducibility | Date Submitted | Last Update | |||
| defect | [Openbravo ERP] A. Platform | major | always | 2017-08-23 18:52 | 2017-09-21 16:49 | |||
| Reporter | dbaz | View Status | public | |||||
| Assigned To | dbaz | |||||||
| Priority | urgent | Resolution | fixed | Fixed in Version | 3.0PR17Q4 | |||
| Status | closed | Fix in branch | Fixed in SCM revision | cc4fe6664e1b | ||||
| Projection | none | ETA | none | Target Version | ||||
| OS | Any | Database | Any | Java version | ||||
| OS Version | Database version | Ant version | ||||||
| Product Version | SCM revision | |||||||
| Merge Request Status | ||||||||
| Review Assigned To | caristu | |||||||
| OBNetwork customer | No | |||||||
| Web browser | ||||||||
| Modules | Core | |||||||
| Support ticket | ||||||||
| Regression level | ||||||||
| Regression date | ||||||||
| Regression introduced in release | ||||||||
| Regression introduced by commit | ||||||||
| Triggers an Emergency Pack | No | |||||||
| Summary | 0036708: CSSMinimizer does not work well with "transform-origin" property | |||||||
| Description | CSSMinimizer does not work well with "transform-origin" property This property can have up three arguments (one per axis) https://www.w3schools.com/cssref/css3_pr_transform-origin.asp [^] The default value is "50% 50% 0". CSSMinimizer has a feature to remove duplicated parameters if exist. This works ok with most of the properties, e.g. "margin", where "100% 100%" or "100% 100% 100% 100%" can be simplified to just "100%", but in the case of "transform-origin", having only one parameter does not assume that the remaining ones have the same value, so this simplification cannot be performed, and all the parameters should be declared in a explicit way (otherwise, the missing one will adopt the default value). This issue has been reported to the owner of the CSSMin library: https://github.com/barryvan/CSSMin/issues/21 [^] | |||||||
| Steps To Reproduce | Include in any CSS loaded within the application. transform-origin: 100% 100%; With the modules NOT in development, notice that the compressed CSS has only "transform-origin: 100%;". (As explained before, this simplification is equivalent to "transform-origin: 100% 50% 0") | |||||||
| Proposed Solution | Avoid the function "simplifyParameters" of CSSMinimizer be executed if the property is "transform-origin" | |||||||
| Tags | No tags attached. | |||||||
| Attached Files | ||||||||
 Relationships [ Relation Graph ]
[ Dependency Graph ]
|
|||||||||||||||||||||||||||||||||
|
|||||||||||||||||||||||||||||||||
 Relationships |
|
Notes |
|
|
(0098620) hgbot (developer) 2017-08-23 18:56 |
Repository: erp/devel/pi Changeset: cc4fe6664e1b584b212307032a5d7096dda5842a Author: David Baz Fayos <david.baz <at> openbravo.com> Date: Wed Aug 23 18:56:08 2017 +0200 URL: http://code.openbravo.com/erp/devel/pi/rev/cc4fe6664e1b584b212307032a5d7096dda5842a [^] Fixed issue 36708: 'transform-origin' css property now is properly minimized --- M modules/org.openbravo.client.kernel/src/org/openbravo/client/kernel/CSSMinimizer.java --- |
|
(0098624) caristu (viewer) 2017-08-24 09:09 |
Code reviewed + tested OK. |
|
(0098816) caristu (viewer) 2017-09-04 16:06 |
Created a pull request[1] in owner's library. [1] https://github.com/barryvan/CSSMin/pull/22 [^] |
|
(0099321) hudsonbot (viewer) 2017-09-21 16:49 |
A changeset related to this issue has been promoted main and to the Central Repository, after passing a series of tests. Promotion changeset: https://code.openbravo.com/erp/devel/main/rev/9750b78d3e5c [^] Maturity status: Test |
|
Notes |
|
Issue History |
|||
| Date Modified | Username | Field | Change |
| 2017-08-23 18:52 | dbaz | New Issue | |
| 2017-08-23 18:52 | dbaz | Assigned To | => dbaz |
| 2017-08-23 18:52 | dbaz | OBNetwork customer | => No |
| 2017-08-23 18:52 | dbaz | Modules | => Core |
| 2017-08-23 18:52 | dbaz | Triggers an Emergency Pack | => No |
| 2017-08-23 18:52 | dbaz | Review Assigned To | => caristu |
| 2017-08-23 18:56 | hgbot | Checkin | |
| 2017-08-23 18:56 | hgbot | Note Added: 0098620 | |
| 2017-08-23 18:56 | hgbot | Status | new => resolved |
| 2017-08-23 18:56 | hgbot | Resolution | open => fixed |
| 2017-08-23 18:56 | hgbot | Fixed in SCM revision | => http://code.openbravo.com/erp/devel/pi/rev/cc4fe6664e1b584b212307032a5d7096dda5842a [^] |
| 2017-08-24 08:31 | caristu | Relationship added | related to 0036683 |
| 2017-08-24 09:09 | caristu | Note Added: 0098624 | |
| 2017-08-24 09:09 | caristu | Status | resolved => closed |
| 2017-08-24 09:09 | caristu | Fixed in Version | => 3.0PR17Q4 |
| 2017-08-24 11:13 | dbaz | Issue cloned | 0036710 |
| 2017-08-24 11:27 | dbaz | Relationship added | related to 0036710 |
| 2017-08-24 12:30 | caristu | Description Updated | View Revisions |
| 2017-08-24 12:31 | caristu | Description Updated | View Revisions |
| 2017-08-25 18:05 | dbaz | Issue cloned | 0036724 |
| 2017-08-25 18:05 | dbaz | Relationship added | related to 0036724 |
| 2017-08-28 09:56 | alostale | Relationship added | related to 0036014 |
| 2017-09-04 16:06 | caristu | Note Added: 0098816 | |
| 2017-09-21 16:49 | hudsonbot | Checkin | |
| 2017-09-21 16:49 | hudsonbot | Note Added: 0099321 | |
|
Issue History |
| Copyright © 2000 - 2009 MantisBT Group |
 |