CSS Layout and Other Cleanups in the Manager Applications

Previous Topic Next Topic
 
classic Classic list List threaded Threaded
21 messages Options
12
Reply | Threaded
Open this post in threaded view
|

CSS Layout and Other Cleanups in the Manager Applications

Jacopo Cappellato
Following the great advices in
https://issues.apache.org/jira/browse/OFBIZ-241 I've started to play
with the styles in maincss.css file and apply the mods to the
manufacturing application.

I've noticed some minor side effects (for example the texts in the main
webtools page are smaller now) and I wanted to be sure that the work I'm
doing is correct and, most of all, if it is a good moment to do this
work now (i.e. before a release).
So please, have a look at the revisions 492187, 492201, 492242
(especially at the changes I did to the maincss.css file) and let me
know what you think.

Jacopo
Reply | Threaded
Open this post in threaded view
|

Re: CSS Layout and Other Cleanups in the Manager Applications

Adrian Crum
Jacopo,

While we're on the subject, could we also discuss the possibility of combining
the maincss.css and tabstyles.css files into one file?


Jacopo Cappellato wrote:

> Following the great advices in
> https://issues.apache.org/jira/browse/OFBIZ-241 I've started to play
> with the styles in maincss.css file and apply the mods to the
> manufacturing application.
>
> I've noticed some minor side effects (for example the texts in the main
> webtools page are smaller now) and I wanted to be sure that the work I'm
> doing is correct and, most of all, if it is a good moment to do this
> work now (i.e. before a release).
> So please, have a look at the revisions 492187, 492201, 492242
> (especially at the changes I did to the maincss.css file) and let me
> know what you think.
>
> Jacopo
>
Reply | Threaded
Open this post in threaded view
|

Re: CSS Layout and Other Cleanups in the Manager Applications

cjhowe
In reply to this post by Jacopo Cappellato
This is a vague recollection of things read/done on
css. Instead of setting the font size in pixels, you
"should" use em units.  The most common approach is
the following:

BODY {
font-size: 62.5%;
}

This makes 16px = 10px and so now:
0.8 em = 8 px
1 em = 10px
1.2 em = 12 px
1.4 em = 14 px

and so forth.  em instead of pixels is used because
IE/Win doesn't allow the user to resize text that is
set in px.  This may or may not have been
fixed/changed in IE7, but it's a much easier
convention to apply consistently than is pixels.



--- Jacopo Cappellato <[hidden email]> wrote:

> Following the great advices in
> https://issues.apache.org/jira/browse/OFBIZ-241 I've
> started to play
> with the styles in maincss.css file and apply the
> mods to the
> manufacturing application.
>
> I've noticed some minor side effects (for example
> the texts in the main
> webtools page are smaller now) and I wanted to be
> sure that the work I'm
> doing is correct and, most of all, if it is a good
> moment to do this
> work now (i.e. before a release).
> So please, have a look at the revisions 492187,
> 492201, 492242
> (especially at the changes I did to the maincss.css
> file) and let me
> know what you think.
>
> Jacopo
>

Reply | Threaded
Open this post in threaded view
|

Re: CSS Layout and Other Cleanups in the Manager Applications

cjhowe
In reply to this post by Adrian Crum
What benefit do you see in doing this?


--- Adrian Crum <[hidden email]> wrote:

> Jacopo,
>
> While we're on the subject, could we also discuss
> the possibility of combining
> the maincss.css and tabstyles.css files into one
> file?
>
>
> Jacopo Cappellato wrote:
> > Following the great advices in
> > https://issues.apache.org/jira/browse/OFBIZ-241
> I've started to play
> > with the styles in maincss.css file and apply the
> mods to the
> > manufacturing application.
> >
> > I've noticed some minor side effects (for example
> the texts in the main
> > webtools page are smaller now) and I wanted to be
> sure that the work I'm
> > doing is correct and, most of all, if it is a good
> moment to do this
> > work now (i.e. before a release).
> > So please, have a look at the revisions 492187,
> 492201, 492242
> > (especially at the changes I did to the
> maincss.css file) and let me
> > know what you think.
> >
> > Jacopo
> >
>

Reply | Threaded
Open this post in threaded view
|

Re: CSS Layout and Other Cleanups in the Manager Applications

Adrian Crum
Maintaining a single css file instead of two. All components reference both
files, so there's no sense to keep them separate.


Chris Howe wrote:

> What benefit do you see in doing this?
>
>
> --- Adrian Crum <[hidden email]> wrote:
>
>
>>Jacopo,
>>
>>While we're on the subject, could we also discuss
>>the possibility of combining
>>the maincss.css and tabstyles.css files into one
>>file?
>>
>>
>>Jacopo Cappellato wrote:
>>
>>>Following the great advices in
>>>https://issues.apache.org/jira/browse/OFBIZ-241
>>
>>I've started to play
>>
>>>with the styles in maincss.css file and apply the
>>
>>mods to the
>>
>>>manufacturing application.
>>>
>>>I've noticed some minor side effects (for example
>>
>>the texts in the main
>>
>>>webtools page are smaller now) and I wanted to be
>>
>>sure that the work I'm
>>
>>>doing is correct and, most of all, if it is a good
>>
>>moment to do this
>>
>>>work now (i.e. before a release).
>>>So please, have a look at the revisions 492187,
>>
>>492201, 492242
>>
>>>(especially at the changes I did to the
>>
>>maincss.css file) and let me
>>
>>>know what you think.
>>>
>>>Jacopo
>>>
>>
>
>
Reply | Threaded
Open this post in threaded view
|

Re: CSS Layout and Other Cleanups in the Manager Applications

cjhowe
Having them separated allows the flexibility to use a
different style sheet if desired without needing to
assign properties for everything that the stylesheet
being replaced has declared for a particular class or
id.  

All back end components utilize both because most were
developed by copying another component, not because of
a decision that they _should look the same. I think
Open Source Strategies CRM component testifies to that
(although they may be simply using the cascading
effect of css as opposed to a stylesheet replacement).
 Personally, I would prefer additional separation of
maincss.css into the groups that are divided by
comments, but it's a rather trivial distinction.



--- Adrian Crum <[hidden email]> wrote:

> Maintaining a single css file instead of two. All
> components reference both
> files, so there's no sense to keep them separate.
>
>
> Chris Howe wrote:
>
> > What benefit do you see in doing this?
> >
> >
> > --- Adrian Crum <[hidden email]> wrote:
> >
> >
> >>Jacopo,
> >>
> >>While we're on the subject, could we also discuss
> >>the possibility of combining
> >>the maincss.css and tabstyles.css files into one
> >>file?
> >>
> >>
> >>Jacopo Cappellato wrote:
> >>
> >>>Following the great advices in
> >>>https://issues.apache.org/jira/browse/OFBIZ-241
> >>
> >>I've started to play
> >>
> >>>with the styles in maincss.css file and apply the
> >>
> >>mods to the
> >>
> >>>manufacturing application.
> >>>
> >>>I've noticed some minor side effects (for example
> >>
> >>the texts in the main
> >>
> >>>webtools page are smaller now) and I wanted to be
> >>
> >>sure that the work I'm
> >>
> >>>doing is correct and, most of all, if it is a
> good
> >>
> >>moment to do this
> >>
> >>>work now (i.e. before a release).
> >>>So please, have a look at the revisions 492187,
> >>
> >>492201, 492242
> >>
> >>>(especially at the changes I did to the
> >>
> >>maincss.css file) and let me
> >>
> >>>know what you think.
> >>>
> >>>Jacopo
> >>>
> >>
> >
> >
>

Reply | Threaded
Open this post in threaded view
|

Re: CSS Layout and Other Cleanups in the Manager Applications

David E Jones-2

On Jan 3, 2007, at 12:36 PM, Chris Howe wrote:

> Having them separated allows the flexibility to use a
> different style sheet if desired without needing to
> assign properties for everything that the stylesheet
> being replaced has declared for a particular class or
> id.
>
> All back end components utilize both because most were
> developed by copying another component, not because of
> a decision that they _should look the same. I think
> Open Source Strategies CRM component testifies to that
> (although they may be simply using the cascading
> effect of css as opposed to a stylesheet replacement).

Actually no, the back-end applications use the same styles because  
they ARE meant to look the same.

The OpenSourceStrategies stuff is not part of OFBiz and they  
certainly can (and obviously have) introduced their own patterns.

>  Personally, I would prefer additional separation of
> maincss.css into the groups that are divided by
> comments, but it's a rather trivial distinction.

I agree, it is rather trivial. I don't see any problem with combining  
these files.

-David



> --- Adrian Crum <[hidden email]> wrote:
>
>> Maintaining a single css file instead of two. All
>> components reference both
>> files, so there's no sense to keep them separate.
>>
>>
>> Chris Howe wrote:
>>
>>> What benefit do you see in doing this?
>>>
>>>
>>> --- Adrian Crum <[hidden email]> wrote:
>>>
>>>
>>>> Jacopo,
>>>>
>>>> While we're on the subject, could we also discuss
>>>> the possibility of combining
>>>> the maincss.css and tabstyles.css files into one
>>>> file?
>>>>
>>>>
>>>> Jacopo Cappellato wrote:
>>>>
>>>>> Following the great advices in
>>>>> https://issues.apache.org/jira/browse/OFBIZ-241
>>>>
>>>> I've started to play
>>>>
>>>>> with the styles in maincss.css file and apply the
>>>>
>>>> mods to the
>>>>
>>>>> manufacturing application.
>>>>>
>>>>> I've noticed some minor side effects (for example
>>>>
>>>> the texts in the main
>>>>
>>>>> webtools page are smaller now) and I wanted to be
>>>>
>>>> sure that the work I'm
>>>>
>>>>> doing is correct and, most of all, if it is a
>> good
>>>>
>>>> moment to do this
>>>>
>>>>> work now (i.e. before a release).
>>>>> So please, have a look at the revisions 492187,
>>>>
>>>> 492201, 492242
>>>>
>>>>> (especially at the changes I did to the
>>>>
>>>> maincss.css file) and let me
>>>>
>>>>> know what you think.
>>>>>
>>>>> Jacopo
>>>>>
>>>>
>>>
>>>
>>
>

Reply | Threaded
Open this post in threaded view
|

Re: CSS Layout and Other Cleanups in the Manager Applications

Adrian Crum
Thanks David.

I remember you mentioning some time ago your desire for the UI to be "more Zen
Garden-esque." More than likely you were referring to the eCommerce component,
but I don't see why the same can't be true for the rest of OFBiz. At the time
you suggested it, I pictured these two files being combined and - as Chris has
suggested - clear comments inserted to assist developers with modifying OFBiz's
appearance.


David E Jones wrote:

>
> On Jan 3, 2007, at 12:36 PM, Chris Howe wrote:
>
>> Having them separated allows the flexibility to use a
>> different style sheet if desired without needing to
>> assign properties for everything that the stylesheet
>> being replaced has declared for a particular class or
>> id.
>>
>> All back end components utilize both because most were
>> developed by copying another component, not because of
>> a decision that they _should look the same. I think
>> Open Source Strategies CRM component testifies to that
>> (although they may be simply using the cascading
>> effect of css as opposed to a stylesheet replacement).
>
>
> Actually no, the back-end applications use the same styles because  they
> ARE meant to look the same.
>
> The OpenSourceStrategies stuff is not part of OFBiz and they  certainly
> can (and obviously have) introduced their own patterns.
>
>>  Personally, I would prefer additional separation of
>> maincss.css into the groups that are divided by
>> comments, but it's a rather trivial distinction.
>
>
> I agree, it is rather trivial. I don't see any problem with combining  
> these files.
>
> -David
>
>
>
>> --- Adrian Crum <[hidden email]> wrote:
>>
>>> Maintaining a single css file instead of two. All
>>> components reference both
>>> files, so there's no sense to keep them separate.
>>>
>>>
>>> Chris Howe wrote:
>>>
>>>> What benefit do you see in doing this?
>>>>
>>>>
>>>> --- Adrian Crum <[hidden email]> wrote:
>>>>
>>>>
>>>>> Jacopo,
>>>>>
>>>>> While we're on the subject, could we also discuss
>>>>> the possibility of combining
>>>>> the maincss.css and tabstyles.css files into one
>>>>> file?
>>>>>
>>>>>
>>>>> Jacopo Cappellato wrote:
>>>>>
>>>>>> Following the great advices in
>>>>>> https://issues.apache.org/jira/browse/OFBIZ-241
>>>>>
>>>>>
>>>>> I've started to play
>>>>>
>>>>>> with the styles in maincss.css file and apply the
>>>>>
>>>>>
>>>>> mods to the
>>>>>
>>>>>> manufacturing application.
>>>>>>
>>>>>> I've noticed some minor side effects (for example
>>>>>
>>>>>
>>>>> the texts in the main
>>>>>
>>>>>> webtools page are smaller now) and I wanted to be
>>>>>
>>>>>
>>>>> sure that the work I'm
>>>>>
>>>>>> doing is correct and, most of all, if it is a
>>>
>>> good
>>>
>>>>>
>>>>> moment to do this
>>>>>
>>>>>> work now (i.e. before a release).
>>>>>> So please, have a look at the revisions 492187,
>>>>>
>>>>>
>>>>> 492201, 492242
>>>>>
>>>>>> (especially at the changes I did to the
>>>>>
>>>>>
>>>>> maincss.css file) and let me
>>>>>
>>>>>> know what you think.
>>>>>>
>>>>>> Jacopo
>>>>>>
>>>>>
>>>>
>>>>
>>>
>>
>
>
Reply | Threaded
Open this post in threaded view
|

Re: CSS Layout and Other Cleanups in the Manager Applications

David E Jones-2

Yeah, ecommerce has always been the highest priority in my mind but  
improving styling and L&F in general on the back-end would also be  
great.

-David


On Jan 3, 2007, at 1:05 PM, Adrian Crum wrote:

> Thanks David.
>
> I remember you mentioning some time ago your desire for the UI to  
> be "more Zen Garden-esque." More than likely you were referring to  
> the eCommerce component, but I don't see why the same can't be true  
> for the rest of OFBiz. At the time you suggested it, I pictured  
> these two files being combined and - as Chris has suggested - clear  
> comments inserted to assist developers with modifying OFBiz's  
> appearance.
>
>
> David E Jones wrote:
>
>> On Jan 3, 2007, at 12:36 PM, Chris Howe wrote:
>>> Having them separated allows the flexibility to use a
>>> different style sheet if desired without needing to
>>> assign properties for everything that the stylesheet
>>> being replaced has declared for a particular class or
>>> id.
>>>
>>> All back end components utilize both because most were
>>> developed by copying another component, not because of
>>> a decision that they _should look the same. I think
>>> Open Source Strategies CRM component testifies to that
>>> (although they may be simply using the cascading
>>> effect of css as opposed to a stylesheet replacement).
>> Actually no, the back-end applications use the same styles  
>> because  they ARE meant to look the same.
>> The OpenSourceStrategies stuff is not part of OFBiz and they  
>> certainly can (and obviously have) introduced their own patterns.
>>>  Personally, I would prefer additional separation of
>>> maincss.css into the groups that are divided by
>>> comments, but it's a rather trivial distinction.
>> I agree, it is rather trivial. I don't see any problem with  
>> combining  these files.
>> -David
>>> --- Adrian Crum <[hidden email]> wrote:
>>>
>>>> Maintaining a single css file instead of two. All
>>>> components reference both
>>>> files, so there's no sense to keep them separate.
>>>>
>>>>
>>>> Chris Howe wrote:
>>>>
>>>>> What benefit do you see in doing this?
>>>>>
>>>>>
>>>>> --- Adrian Crum <[hidden email]> wrote:
>>>>>
>>>>>
>>>>>> Jacopo,
>>>>>>
>>>>>> While we're on the subject, could we also discuss
>>>>>> the possibility of combining
>>>>>> the maincss.css and tabstyles.css files into one
>>>>>> file?
>>>>>>
>>>>>>
>>>>>> Jacopo Cappellato wrote:
>>>>>>
>>>>>>> Following the great advices in
>>>>>>> https://issues.apache.org/jira/browse/OFBIZ-241
>>>>>>
>>>>>>
>>>>>> I've started to play
>>>>>>
>>>>>>> with the styles in maincss.css file and apply the
>>>>>>
>>>>>>
>>>>>> mods to the
>>>>>>
>>>>>>> manufacturing application.
>>>>>>>
>>>>>>> I've noticed some minor side effects (for example
>>>>>>
>>>>>>
>>>>>> the texts in the main
>>>>>>
>>>>>>> webtools page are smaller now) and I wanted to be
>>>>>>
>>>>>>
>>>>>> sure that the work I'm
>>>>>>
>>>>>>> doing is correct and, most of all, if it is a
>>>>
>>>> good
>>>>
>>>>>>
>>>>>> moment to do this
>>>>>>
>>>>>>> work now (i.e. before a release).
>>>>>>> So please, have a look at the revisions 492187,
>>>>>>
>>>>>>
>>>>>> 492201, 492242
>>>>>>
>>>>>>> (especially at the changes I did to the
>>>>>>
>>>>>>
>>>>>> maincss.css file) and let me
>>>>>>
>>>>>>> know what you think.
>>>>>>>
>>>>>>> Jacopo
>>>>>>>
>>>>>>
>>>>>
>>>>>
>>>>
>>>

Reply | Threaded
Open this post in threaded view
|

Re: CSS Layout and Other Cleanups in the Manager Applications

Tim Ruppert
I have a good start on cleaning up the backend to look just like the  
existing site as an intermediate solution.  I'll do my best to clean  
it up a bit and get that out, but I'm swamped at the moment.

I had also identified a LARGE number of places where styles were  
embedded in the code.  This is definitely something I'd love to help  
out with.

Cheers,
Tim
--
Tim Ruppert
HotWax Media
http://www.hotwaxmedia.com

o:801.649.6594
f:801.649.6595


On Jan 3, 2007, at 4:49 PM, David E Jones wrote:

>
> Yeah, ecommerce has always been the highest priority in my mind but  
> improving styling and L&F in general on the back-end would also be  
> great.
>
> -David
>
>
> On Jan 3, 2007, at 1:05 PM, Adrian Crum wrote:
>
>> Thanks David.
>>
>> I remember you mentioning some time ago your desire for the UI to  
>> be "more Zen Garden-esque." More than likely you were referring to  
>> the eCommerce component, but I don't see why the same can't be  
>> true for the rest of OFBiz. At the time you suggested it, I  
>> pictured these two files being combined and - as Chris has  
>> suggested - clear comments inserted to assist developers with  
>> modifying OFBiz's appearance.
>>
>>
>> David E Jones wrote:
>>
>>> On Jan 3, 2007, at 12:36 PM, Chris Howe wrote:
>>>> Having them separated allows the flexibility to use a
>>>> different style sheet if desired without needing to
>>>> assign properties for everything that the stylesheet
>>>> being replaced has declared for a particular class or
>>>> id.
>>>>
>>>> All back end components utilize both because most were
>>>> developed by copying another component, not because of
>>>> a decision that they _should look the same. I think
>>>> Open Source Strategies CRM component testifies to that
>>>> (although they may be simply using the cascading
>>>> effect of css as opposed to a stylesheet replacement).
>>> Actually no, the back-end applications use the same styles  
>>> because  they ARE meant to look the same.
>>> The OpenSourceStrategies stuff is not part of OFBiz and they  
>>> certainly can (and obviously have) introduced their own patterns.
>>>>  Personally, I would prefer additional separation of
>>>> maincss.css into the groups that are divided by
>>>> comments, but it's a rather trivial distinction.
>>> I agree, it is rather trivial. I don't see any problem with  
>>> combining  these files.
>>> -David
>>>> --- Adrian Crum <[hidden email]> wrote:
>>>>
>>>>> Maintaining a single css file instead of two. All
>>>>> components reference both
>>>>> files, so there's no sense to keep them separate.
>>>>>
>>>>>
>>>>> Chris Howe wrote:
>>>>>
>>>>>> What benefit do you see in doing this?
>>>>>>
>>>>>>
>>>>>> --- Adrian Crum <[hidden email]> wrote:
>>>>>>
>>>>>>
>>>>>>> Jacopo,
>>>>>>>
>>>>>>> While we're on the subject, could we also discuss
>>>>>>> the possibility of combining
>>>>>>> the maincss.css and tabstyles.css files into one
>>>>>>> file?
>>>>>>>
>>>>>>>
>>>>>>> Jacopo Cappellato wrote:
>>>>>>>
>>>>>>>> Following the great advices in
>>>>>>>> https://issues.apache.org/jira/browse/OFBIZ-241
>>>>>>>
>>>>>>>
>>>>>>> I've started to play
>>>>>>>
>>>>>>>> with the styles in maincss.css file and apply the
>>>>>>>
>>>>>>>
>>>>>>> mods to the
>>>>>>>
>>>>>>>> manufacturing application.
>>>>>>>>
>>>>>>>> I've noticed some minor side effects (for example
>>>>>>>
>>>>>>>
>>>>>>> the texts in the main
>>>>>>>
>>>>>>>> webtools page are smaller now) and I wanted to be
>>>>>>>
>>>>>>>
>>>>>>> sure that the work I'm
>>>>>>>
>>>>>>>> doing is correct and, most of all, if it is a
>>>>>
>>>>> good
>>>>>
>>>>>>>
>>>>>>> moment to do this
>>>>>>>
>>>>>>>> work now (i.e. before a release).
>>>>>>>> So please, have a look at the revisions 492187,
>>>>>>>
>>>>>>>
>>>>>>> 492201, 492242
>>>>>>>
>>>>>>>> (especially at the changes I did to the
>>>>>>>
>>>>>>>
>>>>>>> maincss.css file) and let me
>>>>>>>
>>>>>>>> know what you think.
>>>>>>>>
>>>>>>>> Jacopo
>>>>>>>>
>>>>>>>
>>>>>>
>>>>>>
>>>>>
>>>>
>

Reply | Threaded
Open this post in threaded view
|

Re: CSS Layout and Other Cleanups in the Manager Applications

Adrian Crum
I'd like to help with that effort. From my perspective, I see it happening kinda
like this:

1. Combine maincss.css and tabstyles.css files into one file.
2. Test css classes on a single component (what Jacopo is doing now), tweak css
class settings.
3. Once css class settings are finalized, go through all widgets and ftl files
to eliminate embedded styles.

I'm not a css wizard, so I can't be of much help with #1 and #2. I can help with
#3 though.



Tim Ruppert wrote:

> I have a good start on cleaning up the backend to look just like the  
> existing site as an intermediate solution.  I'll do my best to clean  it
> up a bit and get that out, but I'm swamped at the moment.
>
> I had also identified a LARGE number of places where styles were  
> embedded in the code.  This is definitely something I'd love to help  
> out with.
>
> Cheers,
> Tim
> --
> Tim Ruppert
> HotWax Media
> http://www.hotwaxmedia.com
>
> o:801.649.6594
> f:801.649.6595
>
>
> On Jan 3, 2007, at 4:49 PM, David E Jones wrote:
>
>>
>> Yeah, ecommerce has always been the highest priority in my mind but  
>> improving styling and L&F in general on the back-end would also be  
>> great.
>>
>> -David
>>
>>
>> On Jan 3, 2007, at 1:05 PM, Adrian Crum wrote:
>>
>>> Thanks David.
>>>
>>> I remember you mentioning some time ago your desire for the UI to  be
>>> "more Zen Garden-esque." More than likely you were referring to  the
>>> eCommerce component, but I don't see why the same can't be  true for
>>> the rest of OFBiz. At the time you suggested it, I  pictured these
>>> two files being combined and - as Chris has  suggested - clear
>>> comments inserted to assist developers with  modifying OFBiz's
>>> appearance.
>>>
>>>
>>> David E Jones wrote:
>>>
>>>> On Jan 3, 2007, at 12:36 PM, Chris Howe wrote:
>>>>
>>>>> Having them separated allows the flexibility to use a
>>>>> different style sheet if desired without needing to
>>>>> assign properties for everything that the stylesheet
>>>>> being replaced has declared for a particular class or
>>>>> id.
>>>>>
>>>>> All back end components utilize both because most were
>>>>> developed by copying another component, not because of
>>>>> a decision that they _should look the same. I think
>>>>> Open Source Strategies CRM component testifies to that
>>>>> (although they may be simply using the cascading
>>>>> effect of css as opposed to a stylesheet replacement).
>>>>
>>>> Actually no, the back-end applications use the same styles  because  
>>>> they ARE meant to look the same.
>>>> The OpenSourceStrategies stuff is not part of OFBiz and they  
>>>> certainly can (and obviously have) introduced their own patterns.
>>>>
>>>>>  Personally, I would prefer additional separation of
>>>>> maincss.css into the groups that are divided by
>>>>> comments, but it's a rather trivial distinction.
>>>>
>>>> I agree, it is rather trivial. I don't see any problem with  
>>>> combining  these files.
>>>> -David
>>>>
>>>>> --- Adrian Crum <[hidden email]> wrote:
>>>>>
>>>>>> Maintaining a single css file instead of two. All
>>>>>> components reference both
>>>>>> files, so there's no sense to keep them separate.
>>>>>>
>>>>>>
>>>>>> Chris Howe wrote:
>>>>>>
>>>>>>> What benefit do you see in doing this?
>>>>>>>
>>>>>>>
>>>>>>> --- Adrian Crum <[hidden email]> wrote:
>>>>>>>
>>>>>>>
>>>>>>>> Jacopo,
>>>>>>>>
>>>>>>>> While we're on the subject, could we also discuss
>>>>>>>> the possibility of combining
>>>>>>>> the maincss.css and tabstyles.css files into one
>>>>>>>> file?
>>>>>>>>
>>>>>>>>
>>>>>>>> Jacopo Cappellato wrote:
>>>>>>>>
>>>>>>>>> Following the great advices in
>>>>>>>>> https://issues.apache.org/jira/browse/OFBIZ-241
>>>>>>>>
>>>>>>>>
>>>>>>>>
>>>>>>>> I've started to play
>>>>>>>>
>>>>>>>>> with the styles in maincss.css file and apply the
>>>>>>>>
>>>>>>>>
>>>>>>>>
>>>>>>>> mods to the
>>>>>>>>
>>>>>>>>> manufacturing application.
>>>>>>>>>
>>>>>>>>> I've noticed some minor side effects (for example
>>>>>>>>
>>>>>>>>
>>>>>>>>
>>>>>>>> the texts in the main
>>>>>>>>
>>>>>>>>> webtools page are smaller now) and I wanted to be
>>>>>>>>
>>>>>>>>
>>>>>>>>
>>>>>>>> sure that the work I'm
>>>>>>>>
>>>>>>>>> doing is correct and, most of all, if it is a
>>>>>>
>>>>>>
>>>>>> good
>>>>>>
>>>>>>>>
>>>>>>>> moment to do this
>>>>>>>>
>>>>>>>>> work now (i.e. before a release).
>>>>>>>>> So please, have a look at the revisions 492187,
>>>>>>>>
>>>>>>>>
>>>>>>>>
>>>>>>>> 492201, 492242
>>>>>>>>
>>>>>>>>> (especially at the changes I did to the
>>>>>>>>
>>>>>>>>
>>>>>>>>
>>>>>>>> maincss.css file) and let me
>>>>>>>>
>>>>>>>>> know what you think.
>>>>>>>>>
>>>>>>>>> Jacopo
>>>>>>>>>
>>>>>>>>
>>>>>>>
>>>>>>>
>>>>>>
>>>>>
>>
>
>
Reply | Threaded
Open this post in threaded view
|

Re: CSS Layout and Other Cleanups in the Manager Applications

Andrew Zeneski
I'm not too sure about combining all styles into one css file.  
Cleaning them up surely would be a good thing, but tabstyles.css was  
designed to handle the tab sections of screens. Only needed to be  
included in screens which use the top level tabs.

Keeping this separate IMO would make changing the look of the tab bar  
easier to find.

Also, once we start to introduce more DHTML CSS styles, not all pages  
will require these. I would expect new css files for these as well.

I'm just afraid that combining all styles into one file will make it  
more complicated to find and customize. Then again, David's comment  
on keeping them separated nicely with comments may accomplish the  
same thing as having properly named separated css files.

Just my two cents.

Andrew

On Jan 3, 2007, at 7:22 PM, Adrian Crum wrote:

> I'd like to help with that effort. From my perspective, I see it  
> happening kinda like this:
>
> 1. Combine maincss.css and tabstyles.css files into one file.
> 2. Test css classes on a single component (what Jacopo is doing  
> now), tweak css class settings.
> 3. Once css class settings are finalized, go through all widgets  
> and ftl files to eliminate embedded styles.
>
> I'm not a css wizard, so I can't be of much help with #1 and #2. I  
> can help with #3 though.
>
>
>
> Tim Ruppert wrote:
>
>> I have a good start on cleaning up the backend to look just like  
>> the  existing site as an intermediate solution.  I'll do my best  
>> to clean  it up a bit and get that out, but I'm swamped at the  
>> moment.
>> I had also identified a LARGE number of places where styles were  
>> embedded in the code.  This is definitely something I'd love to  
>> help  out with.
>> Cheers,
>> Tim
>> --
>> Tim Ruppert
>> HotWax Media
>> http://www.hotwaxmedia.com
>> o:801.649.6594
>> f:801.649.6595
>> On Jan 3, 2007, at 4:49 PM, David E Jones wrote:
>>>
>>> Yeah, ecommerce has always been the highest priority in my mind  
>>> but  improving styling and L&F in general on the back-end would  
>>> also be  great.
>>>
>>> -David
>>>
>>>
>>> On Jan 3, 2007, at 1:05 PM, Adrian Crum wrote:
>>>
>>>> Thanks David.
>>>>
>>>> I remember you mentioning some time ago your desire for the UI  
>>>> to  be "more Zen Garden-esque." More than likely you were  
>>>> referring to  the eCommerce component, but I don't see why the  
>>>> same can't be  true for the rest of OFBiz. At the time you  
>>>> suggested it, I  pictured these two files being combined and -  
>>>> as Chris has  suggested - clear comments inserted to assist  
>>>> developers with  modifying OFBiz's appearance.
>>>>
>>>>
>>>> David E Jones wrote:
>>>>
>>>>> On Jan 3, 2007, at 12:36 PM, Chris Howe wrote:
>>>>>
>>>>>> Having them separated allows the flexibility to use a
>>>>>> different style sheet if desired without needing to
>>>>>> assign properties for everything that the stylesheet
>>>>>> being replaced has declared for a particular class or
>>>>>> id.
>>>>>>
>>>>>> All back end components utilize both because most were
>>>>>> developed by copying another component, not because of
>>>>>> a decision that they _should look the same. I think
>>>>>> Open Source Strategies CRM component testifies to that
>>>>>> (although they may be simply using the cascading
>>>>>> effect of css as opposed to a stylesheet replacement).
>>>>>
>>>>> Actually no, the back-end applications use the same styles  
>>>>> because  they ARE meant to look the same.
>>>>> The OpenSourceStrategies stuff is not part of OFBiz and they    
>>>>> certainly can (and obviously have) introduced their own patterns.
>>>>>
>>>>>>  Personally, I would prefer additional separation of
>>>>>> maincss.css into the groups that are divided by
>>>>>> comments, but it's a rather trivial distinction.
>>>>>
>>>>> I agree, it is rather trivial. I don't see any problem with  
>>>>> combining  these files.
>>>>> -David
>>>>>
>>>>>> --- Adrian Crum <[hidden email]> wrote:
>>>>>>
>>>>>>> Maintaining a single css file instead of two. All
>>>>>>> components reference both
>>>>>>> files, so there's no sense to keep them separate.
>>>>>>>
>>>>>>>
>>>>>>> Chris Howe wrote:
>>>>>>>
>>>>>>>> What benefit do you see in doing this?
>>>>>>>>
>>>>>>>>
>>>>>>>> --- Adrian Crum <[hidden email]> wrote:
>>>>>>>>
>>>>>>>>
>>>>>>>>> Jacopo,
>>>>>>>>>
>>>>>>>>> While we're on the subject, could we also discuss
>>>>>>>>> the possibility of combining
>>>>>>>>> the maincss.css and tabstyles.css files into one
>>>>>>>>> file?
>>>>>>>>>
>>>>>>>>>
>>>>>>>>> Jacopo Cappellato wrote:
>>>>>>>>>
>>>>>>>>>> Following the great advices in
>>>>>>>>>> https://issues.apache.org/jira/browse/OFBIZ-241
>>>>>>>>>
>>>>>>>>>
>>>>>>>>>
>>>>>>>>> I've started to play
>>>>>>>>>
>>>>>>>>>> with the styles in maincss.css file and apply the
>>>>>>>>>
>>>>>>>>>
>>>>>>>>>
>>>>>>>>> mods to the
>>>>>>>>>
>>>>>>>>>> manufacturing application.
>>>>>>>>>>
>>>>>>>>>> I've noticed some minor side effects (for example
>>>>>>>>>
>>>>>>>>>
>>>>>>>>>
>>>>>>>>> the texts in the main
>>>>>>>>>
>>>>>>>>>> webtools page are smaller now) and I wanted to be
>>>>>>>>>
>>>>>>>>>
>>>>>>>>>
>>>>>>>>> sure that the work I'm
>>>>>>>>>
>>>>>>>>>> doing is correct and, most of all, if it is a
>>>>>>>
>>>>>>>
>>>>>>> good
>>>>>>>
>>>>>>>>>
>>>>>>>>> moment to do this
>>>>>>>>>
>>>>>>>>>> work now (i.e. before a release).
>>>>>>>>>> So please, have a look at the revisions 492187,
>>>>>>>>>
>>>>>>>>>
>>>>>>>>>
>>>>>>>>> 492201, 492242
>>>>>>>>>
>>>>>>>>>> (especially at the changes I did to the
>>>>>>>>>
>>>>>>>>>
>>>>>>>>>
>>>>>>>>> maincss.css file) and let me
>>>>>>>>>
>>>>>>>>>> know what you think.
>>>>>>>>>>
>>>>>>>>>> Jacopo
>>>>>>>>>>
>>>>>>>>>
>>>>>>>>
>>>>>>>>
>>>>>>>
>>>>>>
>>>


smime.p7s (3K) Download Attachment
Reply | Threaded
Open this post in threaded view
|

Re: CSS Layout and Other Cleanups in the Manager Applications

Jacques Le Roux
Administrator
I agree, +1

Jacques

----- Original Message -----
From: "Andrew Zeneski" <[hidden email]>
To: <[hidden email]>
Sent: Thursday, January 11, 2007 7:08 AM
Subject: Re: CSS Layout and Other Cleanups in the Manager Applications


> I'm not too sure about combining all styles into one css file.  
> Cleaning them up surely would be a good thing, but tabstyles.css was  
> designed to handle the tab sections of screens. Only needed to be  
> included in screens which use the top level tabs.
>
> Keeping this separate IMO would make changing the look of the tab bar  
> easier to find.
>
> Also, once we start to introduce more DHTML CSS styles, not all pages  
> will require these. I would expect new css files for these as well.
>
> I'm just afraid that combining all styles into one file will make it  
> more complicated to find and customize. Then again, David's comment  
> on keeping them separated nicely with comments may accomplish the  
> same thing as having properly named separated css files.
>
> Just my two cents.
>
> Andrew
>
> On Jan 3, 2007, at 7:22 PM, Adrian Crum wrote:
>
> > I'd like to help with that effort. From my perspective, I see it  
> > happening kinda like this:
> >
> > 1. Combine maincss.css and tabstyles.css files into one file.
> > 2. Test css classes on a single component (what Jacopo is doing  
> > now), tweak css class settings.
> > 3. Once css class settings are finalized, go through all widgets  
> > and ftl files to eliminate embedded styles.
> >
> > I'm not a css wizard, so I can't be of much help with #1 and #2. I  
> > can help with #3 though.
> >
> >
> >
> > Tim Ruppert wrote:
> >
> >> I have a good start on cleaning up the backend to look just like  
> >> the  existing site as an intermediate solution.  I'll do my best  
> >> to clean  it up a bit and get that out, but I'm swamped at the  
> >> moment.
> >> I had also identified a LARGE number of places where styles were  
> >> embedded in the code.  This is definitely something I'd love to  
> >> help  out with.
> >> Cheers,
> >> Tim
> >> --
> >> Tim Ruppert
> >> HotWax Media
> >> http://www.hotwaxmedia.com
> >> o:801.649.6594
> >> f:801.649.6595
> >> On Jan 3, 2007, at 4:49 PM, David E Jones wrote:
> >>>
> >>> Yeah, ecommerce has always been the highest priority in my mind  
> >>> but  improving styling and L&F in general on the back-end would  
> >>> also be  great.
> >>>
> >>> -David
> >>>
> >>>
> >>> On Jan 3, 2007, at 1:05 PM, Adrian Crum wrote:
> >>>
> >>>> Thanks David.
> >>>>
> >>>> I remember you mentioning some time ago your desire for the UI  
> >>>> to  be "more Zen Garden-esque." More than likely you were  
> >>>> referring to  the eCommerce component, but I don't see why the  
> >>>> same can't be  true for the rest of OFBiz. At the time you  
> >>>> suggested it, I  pictured these two files being combined and -  
> >>>> as Chris has  suggested - clear comments inserted to assist  
> >>>> developers with  modifying OFBiz's appearance.
> >>>>
> >>>>
> >>>> David E Jones wrote:
> >>>>
> >>>>> On Jan 3, 2007, at 12:36 PM, Chris Howe wrote:
> >>>>>
> >>>>>> Having them separated allows the flexibility to use a
> >>>>>> different style sheet if desired without needing to
> >>>>>> assign properties for everything that the stylesheet
> >>>>>> being replaced has declared for a particular class or
> >>>>>> id.
> >>>>>>
> >>>>>> All back end components utilize both because most were
> >>>>>> developed by copying another component, not because of
> >>>>>> a decision that they _should look the same. I think
> >>>>>> Open Source Strategies CRM component testifies to that
> >>>>>> (although they may be simply using the cascading
> >>>>>> effect of css as opposed to a stylesheet replacement).
> >>>>>
> >>>>> Actually no, the back-end applications use the same styles  
> >>>>> because  they ARE meant to look the same.
> >>>>> The OpenSourceStrategies stuff is not part of OFBiz and they    
> >>>>> certainly can (and obviously have) introduced their own patterns.
> >>>>>
> >>>>>>  Personally, I would prefer additional separation of
> >>>>>> maincss.css into the groups that are divided by
> >>>>>> comments, but it's a rather trivial distinction.
> >>>>>
> >>>>> I agree, it is rather trivial. I don't see any problem with  
> >>>>> combining  these files.
> >>>>> -David
> >>>>>
> >>>>>> --- Adrian Crum <[hidden email]> wrote:
> >>>>>>
> >>>>>>> Maintaining a single css file instead of two. All
> >>>>>>> components reference both
> >>>>>>> files, so there's no sense to keep them separate.
> >>>>>>>
> >>>>>>>
> >>>>>>> Chris Howe wrote:
> >>>>>>>
> >>>>>>>> What benefit do you see in doing this?
> >>>>>>>>
> >>>>>>>>
> >>>>>>>> --- Adrian Crum <[hidden email]> wrote:
> >>>>>>>>
> >>>>>>>>
> >>>>>>>>> Jacopo,
> >>>>>>>>>
> >>>>>>>>> While we're on the subject, could we also discuss
> >>>>>>>>> the possibility of combining
> >>>>>>>>> the maincss.css and tabstyles.css files into one
> >>>>>>>>> file?
> >>>>>>>>>
> >>>>>>>>>
> >>>>>>>>> Jacopo Cappellato wrote:
> >>>>>>>>>
> >>>>>>>>>> Following the great advices in
> >>>>>>>>>> https://issues.apache.org/jira/browse/OFBIZ-241
> >>>>>>>>>
> >>>>>>>>>
> >>>>>>>>>
> >>>>>>>>> I've started to play
> >>>>>>>>>
> >>>>>>>>>> with the styles in maincss.css file and apply the
> >>>>>>>>>
> >>>>>>>>>
> >>>>>>>>>
> >>>>>>>>> mods to the
> >>>>>>>>>
> >>>>>>>>>> manufacturing application.
> >>>>>>>>>>
> >>>>>>>>>> I've noticed some minor side effects (for example
> >>>>>>>>>
> >>>>>>>>>
> >>>>>>>>>
> >>>>>>>>> the texts in the main
> >>>>>>>>>
> >>>>>>>>>> webtools page are smaller now) and I wanted to be
> >>>>>>>>>
> >>>>>>>>>
> >>>>>>>>>
> >>>>>>>>> sure that the work I'm
> >>>>>>>>>
> >>>>>>>>>> doing is correct and, most of all, if it is a
> >>>>>>>
> >>>>>>>
> >>>>>>> good
> >>>>>>>
> >>>>>>>>>
> >>>>>>>>> moment to do this
> >>>>>>>>>
> >>>>>>>>>> work now (i.e. before a release).
> >>>>>>>>>> So please, have a look at the revisions 492187,
> >>>>>>>>>
> >>>>>>>>>
> >>>>>>>>>
> >>>>>>>>> 492201, 492242
> >>>>>>>>>
> >>>>>>>>>> (especially at the changes I did to the
> >>>>>>>>>
> >>>>>>>>>
> >>>>>>>>>
> >>>>>>>>> maincss.css file) and let me
> >>>>>>>>>
> >>>>>>>>>> know what you think.
> >>>>>>>>>>
> >>>>>>>>>> Jacopo
> >>>>>>>>>>
> >>>>>>>>>
> >>>>>>>>
> >>>>>>>>
> >>>>>>>
> >>>>>>
> >>>
>
>
Reply | Threaded
Open this post in threaded view
|

Re: CSS Layout and Other Cleanups in the Manager Applications

rajsaini
In reply to this post by Andrew Zeneski
One way to go about it is create a one stylesheet for each application
and let this stylesheet import other required stylesheets. This way only
one stylesheet will be linked into the web page.

Thanks,

Raj

Andrew Zeneski wrote:

> I'm not too sure about combining all styles into one css file.
> Cleaning them up surely would be a good thing, but tabstyles.css was
> designed to handle the tab sections of screens. Only needed to be
> included in screens which use the top level tabs.
>
> Keeping this separate IMO would make changing the look of the tab bar
> easier to find.
>
> Also, once we start to introduce more DHTML CSS styles, not all pages
> will require these. I would expect new css files for these as well.
>
> I'm just afraid that combining all styles into one file will make it
> more complicated to find and customize. Then again, David's comment on
> keeping them separated nicely with comments may accomplish the same
> thing as having properly named separated css files.
>
> Just my two cents.
>
> Andrew
>
> On Jan 3, 2007, at 7:22 PM, Adrian Crum wrote:
>
>> I'd like to help with that effort. From my perspective, I see it
>> happening kinda like this:
>>
>> 1. Combine maincss.css and tabstyles.css files into one file.
>> 2. Test css classes on a single component (what Jacopo is doing now),
>> tweak css class settings.
>> 3. Once css class settings are finalized, go through all widgets and
>> ftl files to eliminate embedded styles.
>>
>> I'm not a css wizard, so I can't be of much help with #1 and #2. I
>> can help with #3 though.
>>
>>
>>
>> Tim Ruppert wrote:
>>
>>> I have a good start on cleaning up the backend to look just like
>>> the  existing site as an intermediate solution.  I'll do my best to
>>> clean  it up a bit and get that out, but I'm swamped at the moment.
>>> I had also identified a LARGE number of places where styles were  
>>> embedded in the code.  This is definitely something I'd love to
>>> help  out with.
>>> Cheers,
>>> Tim
>>> --Tim Ruppert
>>> HotWax Media
>>> http://www.hotwaxmedia.com
>>> o:801.649.6594
>>> f:801.649.6595
>>> On Jan 3, 2007, at 4:49 PM, David E Jones wrote:
>>>>
>>>> Yeah, ecommerce has always been the highest priority in my mind
>>>> but  improving styling and L&F in general on the back-end would
>>>> also be  great.
>>>>
>>>> -David
>>>>
>>>>
>>>> On Jan 3, 2007, at 1:05 PM, Adrian Crum wrote:
>>>>
>>>>> Thanks David.
>>>>>
>>>>> I remember you mentioning some time ago your desire for the UI to  
>>>>> be "more Zen Garden-esque." More than likely you were referring
>>>>> to  the eCommerce component, but I don't see why the same can't
>>>>> be  true for the rest of OFBiz. At the time you suggested it, I  
>>>>> pictured these two files being combined and - as Chris has  
>>>>> suggested - clear comments inserted to assist developers with  
>>>>> modifying OFBiz's appearance.
>>>>>
>>>>>
>>>>> David E Jones wrote:
>>>>>
>>>>>> On Jan 3, 2007, at 12:36 PM, Chris Howe wrote:
>>>>>>
>>>>>>> Having them separated allows the flexibility to use a
>>>>>>> different style sheet if desired without needing to
>>>>>>> assign properties for everything that the stylesheet
>>>>>>> being replaced has declared for a particular class or
>>>>>>> id.
>>>>>>>
>>>>>>> All back end components utilize both because most were
>>>>>>> developed by copying another component, not because of
>>>>>>> a decision that they _should look the same. I think
>>>>>>> Open Source Strategies CRM component testifies to that
>>>>>>> (although they may be simply using the cascading
>>>>>>> effect of css as opposed to a stylesheet replacement).
>>>>>>
>>>>>> Actually no, the back-end applications use the same styles  
>>>>>> because  they ARE meant to look the same.
>>>>>> The OpenSourceStrategies stuff is not part of OFBiz and they  
>>>>>> certainly can (and obviously have) introduced their own patterns.
>>>>>>
>>>>>>>  Personally, I would prefer additional separation of
>>>>>>> maincss.css into the groups that are divided by
>>>>>>> comments, but it's a rather trivial distinction.
>>>>>>
>>>>>> I agree, it is rather trivial. I don't see any problem with  
>>>>>> combining  these files.
>>>>>> -David
>>>>>>
>>>>>>> --- Adrian Crum <[hidden email]> wrote:
>>>>>>>
>>>>>>>> Maintaining a single css file instead of two. All
>>>>>>>> components reference both
>>>>>>>> files, so there's no sense to keep them separate.
>>>>>>>>
>>>>>>>>
>>>>>>>> Chris Howe wrote:
>>>>>>>>
>>>>>>>>> What benefit do you see in doing this?
>>>>>>>>>
>>>>>>>>>
>>>>>>>>> --- Adrian Crum <[hidden email]> wrote:
>>>>>>>>>
>>>>>>>>>
>>>>>>>>>> Jacopo,
>>>>>>>>>>
>>>>>>>>>> While we're on the subject, could we also discuss
>>>>>>>>>> the possibility of combining
>>>>>>>>>> the maincss.css and tabstyles.css files into one
>>>>>>>>>> file?
>>>>>>>>>>
>>>>>>>>>>
>>>>>>>>>> Jacopo Cappellato wrote:
>>>>>>>>>>
>>>>>>>>>>> Following the great advices in
>>>>>>>>>>> https://issues.apache.org/jira/browse/OFBIZ-241
>>>>>>>>>>
>>>>>>>>>>
>>>>>>>>>>
>>>>>>>>>> I've started to play
>>>>>>>>>>
>>>>>>>>>>> with the styles in maincss.css file and apply the
>>>>>>>>>>
>>>>>>>>>>
>>>>>>>>>>
>>>>>>>>>> mods to the
>>>>>>>>>>
>>>>>>>>>>> manufacturing application.
>>>>>>>>>>>
>>>>>>>>>>> I've noticed some minor side effects (for example
>>>>>>>>>>
>>>>>>>>>>
>>>>>>>>>>
>>>>>>>>>> the texts in the main
>>>>>>>>>>
>>>>>>>>>>> webtools page are smaller now) and I wanted to be
>>>>>>>>>>
>>>>>>>>>>
>>>>>>>>>>
>>>>>>>>>> sure that the work I'm
>>>>>>>>>>
>>>>>>>>>>> doing is correct and, most of all, if it is a
>>>>>>>>
>>>>>>>>
>>>>>>>> good
>>>>>>>>
>>>>>>>>>>
>>>>>>>>>> moment to do this
>>>>>>>>>>
>>>>>>>>>>> work now (i.e. before a release).
>>>>>>>>>>> So please, have a look at the revisions 492187,
>>>>>>>>>>
>>>>>>>>>>
>>>>>>>>>>
>>>>>>>>>> 492201, 492242
>>>>>>>>>>
>>>>>>>>>>> (especially at the changes I did to the
>>>>>>>>>>
>>>>>>>>>>
>>>>>>>>>>
>>>>>>>>>> maincss.css file) and let me
>>>>>>>>>>
>>>>>>>>>>> know what you think.
>>>>>>>>>>>
>>>>>>>>>>> Jacopo
>>>>>>>>>>>
>>>>>>>>>>
>>>>>>>>>
>>>>>>>>>
>>>>>>>>
>>>>>>>
>>>>
>

Reply | Threaded
Open this post in threaded view
|

Re: CSS Layout and Other Cleanups in the Manager Applications

Vikash Anand-2
In reply to this post by Jacques Le Roux
that is a nice approach, +1

On 1/11/07, Jacques Le Roux <[hidden email]> wrote:

>
> I agree, +1
>
> Jacques
>
> ----- Original Message -----
> From: "Andrew Zeneski" <[hidden email]>
> To: <[hidden email]>
> Sent: Thursday, January 11, 2007 7:08 AM
> Subject: Re: CSS Layout and Other Cleanups in the Manager Applications
>
>
> > I'm not too sure about combining all styles into one css file.
> > Cleaning them up surely would be a good thing, but tabstyles.css was
> > designed to handle the tab sections of screens. Only needed to be
> > included in screens which use the top level tabs.
> >
> > Keeping this separate IMO would make changing the look of the tab bar
> > easier to find.
> >
> > Also, once we start to introduce more DHTML CSS styles, not all pages
> > will require these. I would expect new css files for these as well.
> >
> > I'm just afraid that combining all styles into one file will make it
> > more complicated to find and customize. Then again, David's comment
> > on keeping them separated nicely with comments may accomplish the
> > same thing as having properly named separated css files.
> >
> > Just my two cents.
> >
> > Andrew
> >
> > On Jan 3, 2007, at 7:22 PM, Adrian Crum wrote:
> >
> > > I'd like to help with that effort. From my perspective, I see it
> > > happening kinda like this:
> > >
> > > 1. Combine maincss.css and tabstyles.css files into one file.
> > > 2. Test css classes on a single component (what Jacopo is doing
> > > now), tweak css class settings.
> > > 3. Once css class settings are finalized, go through all widgets
> > > and ftl files to eliminate embedded styles.
> > >
> > > I'm not a css wizard, so I can't be of much help with #1 and #2. I
> > > can help with #3 though.
> > >
> > >
> > >
> > > Tim Ruppert wrote:
> > >
> > >> I have a good start on cleaning up the backend to look just like
> > >> the  existing site as an intermediate solution.  I'll do my best
> > >> to clean  it up a bit and get that out, but I'm swamped at the
> > >> moment.
> > >> I had also identified a LARGE number of places where styles were
> > >> embedded in the code.  This is definitely something I'd love to
> > >> help  out with.
> > >> Cheers,
> > >> Tim
> > >> --
> > >> Tim Ruppert
> > >> HotWax Media
> > >> http://www.hotwaxmedia.com
> > >> o:801.649.6594
> > >> f:801.649.6595
> > >> On Jan 3, 2007, at 4:49 PM, David E Jones wrote:
> > >>>
> > >>> Yeah, ecommerce has always been the highest priority in my mind
> > >>> but  improving styling and L&F in general on the back-end would
> > >>> also be  great.
> > >>>
> > >>> -David
> > >>>
> > >>>
> > >>> On Jan 3, 2007, at 1:05 PM, Adrian Crum wrote:
> > >>>
> > >>>> Thanks David.
> > >>>>
> > >>>> I remember you mentioning some time ago your desire for the UI
> > >>>> to  be "more Zen Garden-esque." More than likely you were
> > >>>> referring to  the eCommerce component, but I don't see why the
> > >>>> same can't be  true for the rest of OFBiz. At the time you
> > >>>> suggested it, I  pictured these two files being combined and -
> > >>>> as Chris has  suggested - clear comments inserted to assist
> > >>>> developers with  modifying OFBiz's appearance.
> > >>>>
> > >>>>
> > >>>> David E Jones wrote:
> > >>>>
> > >>>>> On Jan 3, 2007, at 12:36 PM, Chris Howe wrote:
> > >>>>>
> > >>>>>> Having them separated allows the flexibility to use a
> > >>>>>> different style sheet if desired without needing to
> > >>>>>> assign properties for everything that the stylesheet
> > >>>>>> being replaced has declared for a particular class or
> > >>>>>> id.
> > >>>>>>
> > >>>>>> All back end components utilize both because most were
> > >>>>>> developed by copying another component, not because of
> > >>>>>> a decision that they _should look the same. I think
> > >>>>>> Open Source Strategies CRM component testifies to that
> > >>>>>> (although they may be simply using the cascading
> > >>>>>> effect of css as opposed to a stylesheet replacement).
> > >>>>>
> > >>>>> Actually no, the back-end applications use the same styles
> > >>>>> because  they ARE meant to look the same.
> > >>>>> The OpenSourceStrategies stuff is not part of OFBiz and they
> > >>>>> certainly can (and obviously have) introduced their own patterns.
> > >>>>>
> > >>>>>>  Personally, I would prefer additional separation of
> > >>>>>> maincss.css into the groups that are divided by
> > >>>>>> comments, but it's a rather trivial distinction.
> > >>>>>
> > >>>>> I agree, it is rather trivial. I don't see any problem with
> > >>>>> combining  these files.
> > >>>>> -David
> > >>>>>
> > >>>>>> --- Adrian Crum <[hidden email]> wrote:
> > >>>>>>
> > >>>>>>> Maintaining a single css file instead of two. All
> > >>>>>>> components reference both
> > >>>>>>> files, so there's no sense to keep them separate.
> > >>>>>>>
> > >>>>>>>
> > >>>>>>> Chris Howe wrote:
> > >>>>>>>
> > >>>>>>>> What benefit do you see in doing this?
> > >>>>>>>>
> > >>>>>>>>
> > >>>>>>>> --- Adrian Crum <[hidden email]> wrote:
> > >>>>>>>>
> > >>>>>>>>
> > >>>>>>>>> Jacopo,
> > >>>>>>>>>
> > >>>>>>>>> While we're on the subject, could we also discuss
> > >>>>>>>>> the possibility of combining
> > >>>>>>>>> the maincss.css and tabstyles.css files into one
> > >>>>>>>>> file?
> > >>>>>>>>>
> > >>>>>>>>>
> > >>>>>>>>> Jacopo Cappellato wrote:
> > >>>>>>>>>
> > >>>>>>>>>> Following the great advices in
> > >>>>>>>>>> https://issues.apache.org/jira/browse/OFBIZ-241
> > >>>>>>>>>
> > >>>>>>>>>
> > >>>>>>>>>
> > >>>>>>>>> I've started to play
> > >>>>>>>>>
> > >>>>>>>>>> with the styles in maincss.css file and apply the
> > >>>>>>>>>
> > >>>>>>>>>
> > >>>>>>>>>
> > >>>>>>>>> mods to the
> > >>>>>>>>>
> > >>>>>>>>>> manufacturing application.
> > >>>>>>>>>>
> > >>>>>>>>>> I've noticed some minor side effects (for example
> > >>>>>>>>>
> > >>>>>>>>>
> > >>>>>>>>>
> > >>>>>>>>> the texts in the main
> > >>>>>>>>>
> > >>>>>>>>>> webtools page are smaller now) and I wanted to be
> > >>>>>>>>>
> > >>>>>>>>>
> > >>>>>>>>>
> > >>>>>>>>> sure that the work I'm
> > >>>>>>>>>
> > >>>>>>>>>> doing is correct and, most of all, if it is a
> > >>>>>>>
> > >>>>>>>
> > >>>>>>> good
> > >>>>>>>
> > >>>>>>>>>
> > >>>>>>>>> moment to do this
> > >>>>>>>>>
> > >>>>>>>>>> work now (i.e. before a release).
> > >>>>>>>>>> So please, have a look at the revisions 492187,
> > >>>>>>>>>
> > >>>>>>>>>
> > >>>>>>>>>
> > >>>>>>>>> 492201, 492242
> > >>>>>>>>>
> > >>>>>>>>>> (especially at the changes I did to the
> > >>>>>>>>>
> > >>>>>>>>>
> > >>>>>>>>>
> > >>>>>>>>> maincss.css file) and let me
> > >>>>>>>>>
> > >>>>>>>>>> know what you think.
> > >>>>>>>>>>
> > >>>>>>>>>> Jacopo
> > >>>>>>>>>>
> > >>>>>>>>>
> > >>>>>>>>
> > >>>>>>>>
> > >>>>>>>
> > >>>>>>
> > >>>
> >
> >
>
Reply | Threaded
Open this post in threaded view
|

Re: CSS Layout and Other Cleanups in the Manager Applications

Tim Ruppert
+1
--
Tim Ruppert
HotWax Media

o:801.649.6594
f:801.649.6595


On Jan 11, 2007, at 3:01 AM, Vikash Anand wrote:

that is a nice approach, +1

On 1/11/07, Jacques Le Roux <[hidden email]> wrote:

I agree, +1

Jacques

----- Original Message -----
From: "Andrew Zeneski" <[hidden email]>
Sent: Thursday, January 11, 2007 7:08 AM
Subject: Re: CSS Layout and Other Cleanups in the Manager Applications


> I'm not too sure about combining all styles into one css file.
> Cleaning them up surely would be a good thing, but tabstyles.css was
> designed to handle the tab sections of screens. Only needed to be
> included in screens which use the top level tabs.
>
> Keeping this separate IMO would make changing the look of the tab bar
> easier to find.
>
> Also, once we start to introduce more DHTML CSS styles, not all pages
> will require these. I would expect new css files for these as well.
>
> I'm just afraid that combining all styles into one file will make it
> more complicated to find and customize. Then again, David's comment
> on keeping them separated nicely with comments may accomplish the
> same thing as having properly named separated css files.
>
> Just my two cents.
>
> Andrew
>
> On Jan 3, 2007, at 7:22 PM, Adrian Crum wrote:
>
> > I'd like to help with that effort. From my perspective, I see it
> > happening kinda like this:
> >
> > 1. Combine maincss.css and tabstyles.css files into one file.
> > 2. Test css classes on a single component (what Jacopo is doing
> > now), tweak css class settings.
> > 3. Once css class settings are finalized, go through all widgets
> > and ftl files to eliminate embedded styles.
> >
> > I'm not a css wizard, so I can't be of much help with #1 and #2. I
> > can help with #3 though.
> >
> >
> >
> > Tim Ruppert wrote:
> >
> >> I have a good start on cleaning up the backend to look just like
> >> the  existing site as an intermediate solution.  I'll do my best
> >> to clean  it up a bit and get that out, but I'm swamped at the
> >> moment.
> >> I had also identified a LARGE number of places where styles were
> >> embedded in the code.  This is definitely something I'd love to
> >> help  out with.
> >> Cheers,
> >> Tim
> >> --
> >> Tim Ruppert
> >> HotWax Media
> >> o:801.649.6594
> >> f:801.649.6595
> >> On Jan 3, 2007, at 4:49 PM, David E Jones wrote:
> >>>
> >>> Yeah, ecommerce has always been the highest priority in my mind
> >>> but  improving styling and L&F in general on the back-end would
> >>> also be  great.
> >>>
> >>> -David
> >>>
> >>>
> >>> On Jan 3, 2007, at 1:05 PM, Adrian Crum wrote:
> >>>
> >>>> Thanks David.
> >>>>
> >>>> I remember you mentioning some time ago your desire for the UI
> >>>> to  be "more Zen Garden-esque." More than likely you were
> >>>> referring to  the eCommerce component, but I don't see why the
> >>>> same can't be  true for the rest of OFBiz. At the time you
> >>>> suggested it, I  pictured these two files being combined and -
> >>>> as Chris has  suggested - clear comments inserted to assist
> >>>> developers with  modifying OFBiz's appearance.
> >>>>
> >>>>
> >>>> David E Jones wrote:
> >>>>
> >>>>> On Jan 3, 2007, at 12:36 PM, Chris Howe wrote:
> >>>>>
> >>>>>> Having them separated allows the flexibility to use a
> >>>>>> different style sheet if desired without needing to
> >>>>>> assign properties for everything that the stylesheet
> >>>>>> being replaced has declared for a particular class or
> >>>>>> id.
> >>>>>>
> >>>>>> All back end components utilize both because most were
> >>>>>> developed by copying another component, not because of
> >>>>>> a decision that they _should look the same. I think
> >>>>>> Open Source Strategies CRM component testifies to that
> >>>>>> (although they may be simply using the cascading
> >>>>>> effect of css as opposed to a stylesheet replacement).
> >>>>>
> >>>>> Actually no, the back-end applications use the same styles
> >>>>> because  they ARE meant to look the same.
> >>>>> The OpenSourceStrategies stuff is not part of OFBiz and they
> >>>>> certainly can (and obviously have) introduced their own patterns.
> >>>>>
> >>>>>>  Personally, I would prefer additional separation of
> >>>>>> maincss.css into the groups that are divided by
> >>>>>> comments, but it's a rather trivial distinction.
> >>>>>
> >>>>> I agree, it is rather trivial. I don't see any problem with
> >>>>> combining  these files.
> >>>>> -David
> >>>>>
> >>>>>> --- Adrian Crum <[hidden email]> wrote:
> >>>>>>
> >>>>>>> Maintaining a single css file instead of two. All
> >>>>>>> components reference both
> >>>>>>> files, so there's no sense to keep them separate.
> >>>>>>>
> >>>>>>>
> >>>>>>> Chris Howe wrote:
> >>>>>>>
> >>>>>>>> What benefit do you see in doing this?
> >>>>>>>>
> >>>>>>>>
> >>>>>>>> --- Adrian Crum <[hidden email]> wrote:
> >>>>>>>>
> >>>>>>>>
> >>>>>>>>> Jacopo,
> >>>>>>>>>
> >>>>>>>>> While we're on the subject, could we also discuss
> >>>>>>>>> the possibility of combining
> >>>>>>>>> the maincss.css and tabstyles.css files into one
> >>>>>>>>> file?
> >>>>>>>>>
> >>>>>>>>>
> >>>>>>>>> Jacopo Cappellato wrote:
> >>>>>>>>>
> >>>>>>>>>> Following the great advices in
> >>>>>>>>>
> >>>>>>>>>
> >>>>>>>>>
> >>>>>>>>> I've started to play
> >>>>>>>>>
> >>>>>>>>>> with the styles in maincss.css file and apply the
> >>>>>>>>>
> >>>>>>>>>
> >>>>>>>>>
> >>>>>>>>> mods to the
> >>>>>>>>>
> >>>>>>>>>> manufacturing application.
> >>>>>>>>>>
> >>>>>>>>>> I've noticed some minor side effects (for example
> >>>>>>>>>
> >>>>>>>>>
> >>>>>>>>>
> >>>>>>>>> the texts in the main
> >>>>>>>>>
> >>>>>>>>>> webtools page are smaller now) and I wanted to be
> >>>>>>>>>
> >>>>>>>>>
> >>>>>>>>>
> >>>>>>>>> sure that the work I'm
> >>>>>>>>>
> >>>>>>>>>> doing is correct and, most of all, if it is a
> >>>>>>>
> >>>>>>>
> >>>>>>> good
> >>>>>>>
> >>>>>>>>>
> >>>>>>>>> moment to do this
> >>>>>>>>>
> >>>>>>>>>> work now (i.e. before a release).
> >>>>>>>>>> So please, have a look at the revisions 492187,
> >>>>>>>>>
> >>>>>>>>>
> >>>>>>>>>
> >>>>>>>>> 492201, 492242
> >>>>>>>>>
> >>>>>>>>>> (especially at the changes I did to the
> >>>>>>>>>
> >>>>>>>>>
> >>>>>>>>>
> >>>>>>>>> maincss.css file) and let me
> >>>>>>>>>
> >>>>>>>>>> know what you think.
> >>>>>>>>>>
> >>>>>>>>>> Jacopo
> >>>>>>>>>>
> >>>>>>>>>
> >>>>>>>>
> >>>>>>>>
> >>>>>>>
> >>>>>>
> >>>
>
>



smime.p7s (3K) Download Attachment
Reply | Threaded
Open this post in threaded view
|

Re: CSS Layout and Other Cleanups in the Manager Applications

Adrian Crum
In reply to this post by Andrew Zeneski
Andrew Zeneski wrote:
> I'm not too sure about combining all styles into one css file.  Cleaning
> them up surely would be a good thing, but tabstyles.css was  designed to
> handle the tab sections of screens. Only needed to be  included in
> screens which use the top level tabs.

That's not true. The way OFBiz is set up now, tabstyles.css is used with EVERY
screen. That's why I combined them. If we're referencing both files with every
request, then why have two files at all?

If a component has a unique style sheet, then there are ways to get that added
to the list - via layoutSettings.stylesheets[].

> I'm just afraid that combining all styles into one file will make it  
> more complicated to find and customize. Then again, David's comment  on
> keeping them separated nicely with comments may accomplish the  same
> thing as having properly named separated css files.

How can searching for a specific style be easier with multiple files? With a
single main stylesheet, if you want to change a style you would have to look in
only one place.

Reply | Threaded
Open this post in threaded view
|

Re: CSS Layout and Other Cleanups in the Manager Applications

Andrew Sykes
In reply to this post by Andrew Zeneski
I agree with Andy here.

But I think the tabstyles stuff needs to be clearly marked and separated
within the merged file. So if anyone wants to implement a new set there
is a clearly marked set of classes.

- Andrew

On Thu, 2007-01-11 at 08:15 -0800, Adrian Crum wrote:

> Andrew Zeneski wrote:
> > I'm not too sure about combining all styles into one css file.  Cleaning
> > them up surely would be a good thing, but tabstyles.css was  designed to
> > handle the tab sections of screens. Only needed to be  included in
> > screens which use the top level tabs.
>
> That's not true. The way OFBiz is set up now, tabstyles.css is used with EVERY
> screen. That's why I combined them. If we're referencing both files with every
> request, then why have two files at all?
>
> If a component has a unique style sheet, then there are ways to get that added
> to the list - via layoutSettings.stylesheets[].
>
> > I'm just afraid that combining all styles into one file will make it  
> > more complicated to find and customize. Then again, David's comment  on
> > keeping them separated nicely with comments may accomplish the  same
> > thing as having properly named separated css files.
>
> How can searching for a specific style be easier with multiple files? With a
> single main stylesheet, if you want to change a style you would have to look in
> only one place.
>
--
Kind Regards
Andrew Sykes <[hidden email]>
Sykes Development Ltd
http://www.sykesdevelopment.com

Reply | Threaded
Open this post in threaded view
|

Re: CSS Layout and Other Cleanups in the Manager Applications

Adrian Crum
Andrew Sykes wrote:
> But I think the tabstyles stuff needs to be clearly marked and separated
> within the merged file. So if anyone wants to implement a new set there
> is a clearly marked set of classes.

If you take a look at the patch in
https://issues.apache.org/jira/browse/OFBIZ-605 you'll see that that has been done.


Reply | Threaded
Open this post in threaded view
|

Re: CSS Layout and Other Cleanups in the Manager Applications

Andrew Sykes
In reply to this post by Andrew Sykes
Adrian,

Sorry, I was aware you had done it this way, it wasn't a suggestion so
much as a +1 for the way you had done it.

Of course, now I read it back it's hard to see that this is what I
meant ;-)

- Andrew

On Thu, 2007-01-11 at 12:44 -0800, Adrian Crum wrote:
> Andrew Sykes wrote:
> > But I think the tabstyles stuff needs to be clearly marked and separated
> > within the merged file. So if anyone wants to implement a new set there
> > is a clearly marked set of classes.
>
> If you take a look at the patch in
> https://issues.apache.org/jira/browse/OFBIZ-605 you'll see that that has been done.
>
--
Kind Regards
Andrew Sykes <[hidden email]>
Sykes Development Ltd
http://www.sykesdevelopment.com


12