Sign Up! Login: Password: New User? Forgot? Support
Top Mods!
Wowhead Search
by sonofsamedi
Digital Clock
by Wolfar15
Admin Forums
Welcome to the GuildPortal Help Community! These forums are for admin-to-admin help.
Available Forums
Forums : Feature Discussion > For the love of the internet - Use Divs and CSS!!!
Pinkrocket (New Admin) 10/14/2009 7:08 PM EST : For the love of the internet - Use Divs and CSS!!!

Pinkrocket
Posts: 7
Getting Started!

It would be absolutely wonderful if GuildPortal sites were by default assembled using CSS.

At moment it seems everytime the server comes down something within the layout changes and breaks my design because GP is using tables.

So PLEASE GP EMBRACE CSS!
TopBottom

Aaron Lewis (Blue Collar) 10/14/2009 9:03 PM EST : RE: For the love of the internet - Use Divs and CSS!!!
GuildPortal Dev
Aaron Lewis
Posts: 1979
Zomgawsh Poster

Which element(s) in particular are being a nuisance?  I'll see what I can do.
Aaron Lewis, GuildPortal.com
TopBottom

AbbydonKrafts (New Admin) 2/7/2010 2:10 PM EST : RE: For the love of the internet - Use Divs and CSS!!!
AbbydonKrafts
Posts: 67
Posts With Wolves

I see two problems:
  1. Use of tables where they shouldn't be used. Only tabular data should use Tables.
    If you can add a summary/totals row at the bottom, then you know it's tabular data.
  2. Missing/incorrect class names/IDs.
Here are some examples of #2. Remember that the order of names is important in the class attribute.

The large news box uses "ForumCategoryHeader" as the topic header.

Recommended: "ForumCategoryHeader NewsHeader"
My workaround: ".gpWidget-newslrg .ForumCategoryHeader".

Tradeskills does not have it's own class, only "ContentBoxHeader".

Recommended: "ContentBoxTitle TradeskillHeader"
My workaround: .gpWidget-tradeskills .ContentBoxBody td.ContentBoxTitle

Forum rows are hard to access due to missing granular classes.

Recommended: "ForumRow AltGridRow ForumAltRow"
My workaround: .gpWidget-forumslrg .ContentBoxBody tr[rel="forum"].AltGridRow
That is a bad workaround, btw, due to the TR selector.

Forum sub-categories do not have their own class, only "ForumCategoryHeader".
An example is the "Sticky Posts" row.

Recommended: "ForumCategoryHeader ForumSubcategory"
My workaround: .gpWidget-forumslrg tr.ForumCategoryHeader td[colspan="5"]
This is so it won't affect the "Topics..Replies..etc" header row.

So yes, please assign classes to all major elements in addition to the "communal" ones.
TopBottom

Admins Online
There are   members online.
So-and-so has logged on!
%title%
%message%