[jira] [Resolved] (OFBIZ-7311) Formatting and renaming of the CSS files as per best practices

Previous Topic Next Topic
 
classic Classic list List threaded Threaded
1 message Options
Reply | Threaded
Open this post in threaded view
|

[jira] [Resolved] (OFBIZ-7311) Formatting and renaming of the CSS files as per best practices

Nicolas Malin (Jira)

     [ https://issues.apache.org/jira/browse/OFBIZ-7311?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Pranay Pandey resolved OFBIZ-7311.
----------------------------------
       Resolution: Fixed
    Fix Version/s: Upcoming Branch

Thanks so much @Swapnil M Mane for this contribution towards the betterment of overall readability of CSS files in all the themes.
I request all the developers working on OFBiz Trunk to do ./ant clean-all load-demo after updating code post this revision as renamed CSS references have been updated in theme configuration.

Patches committed to trunk revisions- 1748916, 1748917, 1748918, 1748919, 1748920, 1748921, 1748922, 1748923, 1748924.



> Formatting and renaming of the CSS files as per best practices
> --------------------------------------------------------------
>
>                 Key: OFBIZ-7311
>                 URL: https://issues.apache.org/jira/browse/OFBIZ-7311
>             Project: OFBiz
>          Issue Type: Improvement
>          Components: themes
>    Affects Versions: Trunk
>            Reporter: Swapnil M Mane
>            Assignee: Pranay Pandey
>            Priority: Minor
>             Fix For: Upcoming Branch
>
>         Attachments: OFBIZ-7311-bizznesstime.patch, OFBIZ-7311-bluelight.patch, OFBIZ-7311-droppingcrumbs.patch, OFBIZ-7311-flatgrey.patch, OFBIZ-7311-multiflex.patch, OFBIZ-7311-rainbowstone.patch, OFBIZ-7311-renamed-maincss-to-style.patch, OFBIZ-7311-tomahawk.patch
>
>
> In CSS files, we are having various inconsistency in the formatting of the code for e.g. we have following types of code formatting
> // No space
> {code}
> ul#preferences-menu a:hover {
> text-decoration: none;
> }
> {code}
> // Space of 2
> {code}
> div.autocomplete ul {
>   list-style-type:none;
>   margin:0;
>   padding:0;
> }
> {code}
> // Space of 4
> {code}
> .control-area a {
>     font-size: 1.1em;
>     color: #5CA3D7;
> }
> {code}
> For better readability we should format the existing files with space of 4.
> Also for some theme, we named CSS files as *maincss.css* and for some, we named it as *style.css*. We can rename these file as per best practices for consistency.



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