|
|
#1 |
|
New Member
![]() Join Date: May 2009
Posts: 2
|
Ok, here is my site. I am still tweaking in the coding, so please don't pay attention to the fact that there aren't any meta keywords! haha...
anyway, here it is: www.clsgraphicsonline.com Feedback is appreciated! |
|
|
|
|
|
#2 |
|
New Member
![]() Join Date: May 2009
Posts: 10
|
With those nice nav buttons i would like to see a "smoother" background image than the stripes kind of plays with my eyes a bit.
I would like to see a mission statement as well "We are a...." so i don't have to search for what you do. |
|
|
|
|
|
#3 |
|
Silver Member
![]() |
Avoid images that consist of nothing but text. Your flash on your front page doesn't have working links, a load graphic or appear to be done in flash since it's got a HUGE load size. Layout is ok, and your auto scrolling portfolio aggravates more than showcases your work, look into a simpleviewer flash script if you're gonna have a flash portfolio.
|
|
|
|
|
|
#4 |
|
New Member
![]() Join Date: May 2009
Posts: 2
|
Ok, so if I made the lines blend a little more so it looks more solid do you think that would be better?
Also, thanks for the comment about the images and text! I meant to change that and completely forgot! I have heard so many people say bad things about intro pages, do you think I should go ahead and put a simpler intro page on there to give my home page time to load up? I am not very good in flash myself. So, with work, at least until I get the hang of it better, I subcontract that out. Myself...I am not sure how to make a link in the flash video...I have been trying to figure it out, but I do not know enough about flash to get it right. Definitely not interested in a flash portfolio...I used deamweavers scroll tabs to create that portfolio layout. Maybe I should just list the information without the tabs? Do you think that would work better? |
|
|
|
|
|
#5 |
|
Silver Member
![]() |
well when i do a portfolio or gallery of images i almost exclusively use simpleviewer or i use thumbnails with a lightbox script
you could try a javascript image replacement script, but really anything else annoys people. |
|
|
|
|
|
#6 |
|
Silver Member
![]() Join Date: May 2009
Location: Houston & The Woodlands
Posts: 110
|
No - don't use Javascript for image replacement, there is a MUCH simpler way that doesn't require it at all.
CSS works just fine! Just take your <div> with the image you are going to replace it with- <div class="imageReplacement">Text used in your image goes here!</div> Code:
.imageReplacement {
width: 490px; /* Replace with image width */
height: 0px;
/* Replace H1 Tag with background image */
background: url(/images/blank.png) no-repeat center top;
padding-top: 78px!important; /* same height as your image */
overflow: hidden; margin: 10px auto;
}
To see an example, check out our home page. |
|
|
|
|
|
#7 |
|
Silver Member
![]() |
What you just described was for replacing a piece of text with an image. a javascript image replace script normally replaces one image for another via a link. I used them back in the day, but because people have javascript disabled i stopped using it. Something along the lines of this. Or ... this one is another good example.
|
|
|
|
![]() |
| Tags |
| custom websites, design, feedback, web site |
| Thread Tools | |
| Display Modes | |
|
|