Post by hgecko
Hi everybody! This is my first forum post!
Like everyone else on Wowhead, I've been drooling over the MoP talent calculator. I'm pretty excited about the changes to talents in the new expansion! There are two small things that really drive me nuts about it, though. I'm here to point them out and make suggestions. I really have little experience in web design, but I hope you'll take them into consideration in the future!
Just to be clear, the page I am solely referring to in this post can be found
here.
I am using
Firebug to make the live changes.
Here we go!
Item 1:0_o <--- my face when I first saw this.
I fixed this by adjusting the CSS for this page. Specifically, I changed the CSS style in mop-tc.css
from.tc-classes-inner { width: 460px; }
to.tc-classes-inner { width: 506px; }
Here's a screenshot of the fixed page!Let me know what you think of this! Personally, I think it's a very small change, but totally worth the effort to fix.
Item 2:I fixed this by removing a hadcoded div style for the "mop-spells-table" class:
<div class="mop-spells-table" style="height: 207px;">
I simply removed the "style" portion of that HTML tag. This isn't
entirely necessary, but it keeps things clean and simple for the change in the next step.
I then added a rule to the .mop-spells-table class:
.mop-spells-table { height: 500px; }
Here's a screenshot of the longer spell list!Obviously, that really changes things. It's not very pretty to look at, but it does feel less crammed when you're exploring the very long list of spells every class has! What do you think of this one? I chose 500px as an arbitrary number.
I really enjoy Wowhead, so I'm making a few suggestions for improvements to a feature of the site. Any feedback is appreciated, and of course, I hope the maintainers like my suggestions! If you'd like a
Stylish mockup or something, let me know. It was suggested to me that I make one specifically for this, so I might take a shot at it.
Post by Nulgar
CSS hacking ftw!
Hmm, let me add a minor gripe with Wowhead's CSS as well, or rather, the CSS that the tooltip script injects on sites it is used - well, the same files anyway ;)
I'm using a GreaseMonkey script to have it on every site, so it's actually my own fault, but anyway, CSS classes like "c1" to "c11" are quite ambigous, I've noticed it affecting e.g. the Amazon shopping
cart, where all items are in paladin pink, or
comments below articles on several sites,
example.
I'm not even suggesting to change it simply because of that, but if you ever overhaul/rewrite that system, consider using less ambigous class names :P
