Forum Subject: professional_css
Posted on: 06/11/11 01:59pm
By: ivy
We Geeklog Japanese group changed theme name 'ProfessionsCSS' to 'professional_css'.
And I added style to admin/config/configuration.thtml, because left side bloack is too narrow.
http://code.google.com/p/geeklog-jp/source/browse/trunk/geeklog-jp/public_html/layout/professional_css/admin/config/configuration.thtml[*1]
left: 25% right:75%
and I deleted font-family style on tool tips..
http://code.google.com/p/geeklog-jp/source/browse/trunk/geeklog-jp/public_html/layout/professional_css/jquery_ui/jquery-ui.css[*2]
How about this changes.
Thanks.
And I added style to admin/config/configuration.thtml, because left side bloack is too narrow.
http://code.google.com/p/geeklog-jp/source/browse/trunk/geeklog-jp/public_html/layout/professional_css/admin/config/configuration.thtml[*1]
left: 25% right:75%
and I deleted font-family style on tool tips..
http://code.google.com/p/geeklog-jp/source/browse/trunk/geeklog-jp/public_html/layout/professional_css/jquery_ui/jquery-ui.css[*2]
How about this changes.
Thanks.
Re: professiona_css
Posted on: 06/12/11 09:49am
By: Roccivic
Just pasting this here from our email conversation, as I think that I have a valid point here.
Rouslan
Looks nice, but I'm not sure if it's such a good idea to hardcode widths
into columns. What if there is one really long word in the left column?
It will either get chopped or will overflow into the right frame and
that's not pretty. If you really want to get this change into the theme,
please post about it on the forum or on the devel mailing list, so that
other developers can also have a look and express their opinion.
into columns. What if there is one really long word in the left column?
It will either get chopped or will overflow into the right frame and
that's not pretty. If you really want to get this change into the theme,
please post about it on the forum or on the devel mailing list, so that
other developers can also have a look and express their opinion.
Rouslan
Re: professiona_css
Posted on: 06/12/11 01:19pm
By: Laugh
Hey Ivy,
For any jQuery CSS stuff you should probably update the jquery.ui.geeklog.css file. It just makes updating jQuery easier if I do not have to worry about copying over any changes in those files.
Your updated column sizes look better. The problem of a really long word that Roccivic mentioned can happen but it could potenially happen anyways even with out the widths specified (ie both columns have a really long word) so for myself I am not to worried about that.
As far as inline styles are concerned I know that they are a no no, but I am guilty of this as well. I think since we have several people working on themes we should develop some sort of html/css standard (at least a basic one) like we do for PHP coding.
For example here are a few questions that I have:
1) Inline styles are usually frowned upon but is it "okay" if it is just one or two items (like setting the text align for a table colum) or should we create a class for that
2) How should are css class definitions look? How about spacing? (ie vertical-align:top; or vertical-align: top
3) How should we name classes?
4) etc...
Answering these types of questions will make things a bit easier. As a reference Drupal has started a CSS coding standard http://drupal.org/node/302199[*3]
Tom
and I deleted font-family style on tool tips..
http://code.google.com/p/geeklog-jp/source/browse/trunk/geeklog-jp/public_html/layout/professional_css/jquery_ui/jquery-ui.css
http://code.google.com/p/geeklog-jp/source/browse/trunk/geeklog-jp/public_html/layout/professional_css/jquery_ui/jquery-ui.css
For any jQuery CSS stuff you should probably update the jquery.ui.geeklog.css file. It just makes updating jQuery easier if I do not have to worry about copying over any changes in those files.
And I added style to admin/config/configuration.thtml, because left side bloack is too narrow.
http://code.google.com/p/geeklog-jp/source/browse/trunk/geeklog-jp/public_html/layout/professional_css/admin/config/configuration.thtml
left: 25% right:75%
http://code.google.com/p/geeklog-jp/source/browse/trunk/geeklog-jp/public_html/layout/professional_css/admin/config/configuration.thtml
left: 25% right:75%
Your updated column sizes look better. The problem of a really long word that Roccivic mentioned can happen but it could potenially happen anyways even with out the widths specified (ie both columns have a really long word) so for myself I am not to worried about that.
As far as inline styles are concerned I know that they are a no no, but I am guilty of this as well. I think since we have several people working on themes we should develop some sort of html/css standard (at least a basic one) like we do for PHP coding.
For example here are a few questions that I have:
1) Inline styles are usually frowned upon but is it "okay" if it is just one or two items (like setting the text align for a table colum) or should we create a class for that
2) How should are css class definitions look? How about spacing? (ie vertical-align:top; or vertical-align: top
3) How should we name classes?
4) etc...
Answering these types of questions will make things a bit easier. As a reference Drupal has started a CSS coding standard http://drupal.org/node/302199[*3]
Tom
Re: professional_css
Posted on: 06/12/11 08:14pm
By: ivy
Hard coding "left: 25% right:75% " is better, and css style coding is best I think, too.
Re: professional_css
Posted on: 06/13/11 04:56am
By: Roccivic
Quote by: Laugh
1) Inline styles are usually frowned upon but is it "okay" if it is just one or two items (like setting the text align for a table colum) or should we create a class for that
Hardcoding text-aligns is especially a bad idea, as I think it will compromise RTL support (we have exactly this problem in the forum plugin ATM). Though I'd have no problems with inline delarations for, say, font-weight.
Quote by: Laugh
Answering these types of questions will make things a bit easier. As a reference Drupal has started a CSS coding standard http://drupal.org/node/302199[*3]
It looks good, let's just steal it, like we did with the PEAR standard
How is it that we handle RTL anyway? We don't really have conditional CSS file loading mechanism like in Drupal, right?
Re: professional_css
Posted on: 06/13/11 05:57am
By: Dirk
Quote by: Roccivic
How is it that we handle RTL anyway? We don't really have conditional CSS file loading mechanism like in Drupal, right?
Nope, nothing fancy like this. All you have to go by is the $LANG_DIRECTION variable. I think in some places we set that as a template variable so that you can then use it in CSS class names.
bye, Dirk
Re: professional_css
Posted on: 06/14/11 05:59am
By: ivy
Templates of configuration use table layout.
But I don't like table layout.
Only configuration use table layout in professional_css.
But I don't like table layout.
Only configuration use table layout in professional_css.
Re: professional_css
Posted on: 06/14/11 09:03am
By: Laugh
Quote by: ivy
Templates of configuration use table layout.
But I don't like table layout.
Only configuration use table layout in professional_css.
Yeah, that was something I was going to look into at some point but never got around to it. Your more than welcome to submit a patch for it
In regards to the CSS coding standard, I will add it to the wiki using most of what Drupal has and post a link here for others to contribute.