Jump to content

The "SoulSilver Luigia Theme" Feedback Thread


Do you like the new theme?  

28 members have voted

  1. 1. Do you like the new theme?

    • I love it.
    • I HATE it.
    • I think that it was released too early.
    • I want to stay out of this.
    • My browser work properly with it, so I cannot have an opinion.


Recommended Posts

Hey what's that blue line on the SoulSilver Lugia Theme. I circled it in red:

82275379.png

EDIT: Also, look at the top-right side, the Total Private Messages is on the next line. And its colour is impossible to see, unless I highlight it.

Link to comment
Share on other sites

I like darker themes cause it not only saves power for my computer, it also helps my eyes a bit when i'm up at night looking around :P. But i like it anyway.

But if it were me i would have more of a deep sea thing to darken the colors and also really tell that lugia lives in the ocean.

I remember seeing somewhere (trustworthy) that black actually uses MORE power.

Link to comment
Share on other sites

The only thing I don't like about it is the lack of contrast over the last visited time and pm section.

http://img193.imageshack.us/img193/6285/soulsilverthemeprojectp.jpg

Edit: On smaller resolutions the logout button up top exceeds the container width. On 1024 it spills over

Because the layout is fluid you'll have to adjust the padding to fix it in lower resolutions. I haven't had a look at 800 x 600 though.

Look in #navigation li, because you're using css shorthand you're adding 24 total pixels of padding between the menu links. You're missing including logout by 2-4 pixels so lower it to ten by changing padding: 11px 12px; to padding: 11px 10px;

#navigation li {
background: transparent url(hoiqantinhoc/pig/emu/style/navSplit.gif) no-repeat scroll right top;
float: left;
padding: 11px 10px;
}

Add "color: #fff;" to .memberBox to add contrast to the image above

Edit 2: looks like some change made somewhere is now causing the ad banner to shift down below because it doesn't have enough space to display in it's present condition at the 1024 screen resolution.

.bannerBox {

margin:44px 8px 0 0;

width:487px;

}

This doesn't allow for enough room at 1024, fair warning the fix below won't do much to help at the 800x600 resolution but it won't hurt as it will have to be done anyway when fixing for 800x600. You'll need to set min-widths along the entire container in order to fix the layout at that, but it will cause a scroll bar. Otherwise you're in for a real headache.

I should also note that adding margin to the right on an element that is floated to the right triggers a bug in IE6 causing that 8px to double to 16px. The double margin bug. If you have to add margin right or left to something good practice is to make sure the element you're adding it to isn't floated in that same direction. In such cases use padding as an alternative if possible.

This should fix it for the 1024 resolution

.bannerBox {

margin:44px 0 0 0;

width:480px;

}

//Nearly all of the issues pointed out are things that look fine in resolutions above 1024 but break when they are at those resolutions or lower. You'll want to check statistical data and see what resolutions the viewers of the site are looking at it in. A large percentage may lie in that region.

Edited by DefiantHawk
Link to comment
Share on other sites

  • 3 weeks later...

Create an account or sign in to comment

You need to be a member in order to leave a comment

Create an account

Sign up for a new account in our community. It's easy!

Register a new account

Sign in

Already have an account? Sign in here.

Sign In Now
×
×
  • Create New...