Hello and welcome to our community! Is this your first visit?
Register
Enjoy an ad free experience by logging in. Not a member yet? Register.
Results 1 to 6 of 6
  1. #1
    New Coder
    Join Date
    May 2009
    Posts
    11
    Thanks
    0
    Thanked 0 Times in 0 Posts

    Looking for comments and critique, advice welcome too

    Hello! New member. I recently had to brush up on my HTML skills and grew interested in the subject. Started looking deeper into it and I'm considering a small career shift into web design.

    Anyway, this is the site that got me started: profytec.com.ar.

    A few years ago my dad wanted a small web presence for some projects of his, and being too cheap to hire someone instead asked me to do it. I read up a bit on the subject, and put out a basic web design (think it was around 2000, 2002).

    A few weeks ago he wanted a small update to the page, and being rusty I started going back over the basics, learned of the advances since, and got a bit excited. I finished the update, but with what I've learned since I did it I know it can be a lot better. For starters, it's a table based design. I know, there's a special spot in hell reserved for me for doing that. But then, I didn't want to tear it down and start anew when I did it. Had to get it done quickly. And I wasn't getting paid.

    Leaving that aside, I was curious about what people thought overall of the page, improvements they'd do and the like. As I said, I'm considering moving into web design in a more serious manner. On the plus side, I can pretty much do whatever I want to the site so I have some space to practice. Thinking a little php would really improve manageability of the site (html menus are a pain to update).

    So... yeah. Any replies are welcome. I did just check that all the pages validate as strict XHTML 1.0 at least.

  • #2
    Senior Coder
    Join Date
    Feb 2009
    Location
    Ilkley, West Yorkshire, UK
    Posts
    2,957
    Thanks
    9
    Thanked 727 Times in 721 Posts
    Page validates - good!

    I can't comment on the content as I don't speak Spanish, but a few comments on the layout and colours.

    Even with screen res at 1280 by 1024 I still get a horizontal scrollbar, maybe that could be removed.

    The page is centred in FF3 but not in IE7.

    Colourwise it's very calm and corporate. Maybe a bolder heading across the top of the page would work better, and have the headings for your different pages a bit lower down. This might make a stronger impact.
    Also the footer looks a bit detached from the rest of the page, maybe you could extend your background colour here, or use a different shade.

    The width of the menu on the left changes from page to page - I think it would be better if this was standard across the site. Also the submenu - Inicio, Neustros Clientes etc - doesn't look right to me with such a large indent - maybe some more styling of these links would make it look nicer.

    Finally, maybe lose the big indent on the first line of each of your paragraph text.

    Hope this is useful....

  • #3
    New Coder
    Join Date
    May 2009
    Posts
    11
    Thanks
    0
    Thanked 0 Times in 0 Posts
    Thanks! This does help.

    The content was given to me to put up. What it says isn't so much a concern for me at this stage.

    Odd thing the horizontal scrollbar. That shouldn't be happening (doesn't happen to me with FF3.0.10). Maybe the <hr /> is doing it....

    I checked the centered thing on IE. That's going to have to get fixed.

    I tried to go for a palette that was easy on the eyes. Calm and corporate is about right. Will see about the the headers, see what works. The footer too.

    I thought I had the menu's width fixed. Might be displaying differently on my browser than yours. I'll have to give cross browser compatibility a once over, fix that.

    I think I'll erase the indents. I don't have enough text, I think, to warrant it. And I've seen most sites seem to do away with them altogether.

    Thanks for the comments. They are very useful.

  • #4
    Supreme Master coder! _Aerospace_Eng_'s Avatar
    Join Date
    Dec 2004
    Location
    In a place far, far away...
    Posts
    19,291
    Thanks
    2
    Thanked 1,043 Times in 1,019 Posts
    Looks plain to me. Also why are you using a table for your navigation? Read the link in my sig titled "Why Tables for Layout is Stupid?".
    ||||If you are getting paid to do a job, don't ask for help on it!||||

  • #5
    Senior Coder
    Join Date
    Feb 2009
    Location
    Ilkley, West Yorkshire, UK
    Posts
    2,957
    Thanks
    9
    Thanked 727 Times in 721 Posts
    Odd thing the horizontal scrollbar. That shouldn't be happening (doesn't happen to me with FF3.0.10). Maybe the <hr /> is doing it....
    Sorry, could have been clearer - I get the scrollbar in IE7, not in FF3.

  • #6
    New Coder
    Join Date
    May 2009
    Posts
    11
    Thanks
    0
    Thanked 0 Times in 0 Posts
    Quote Originally Posted by _Aerospace_Eng_ View Post
    Looks plain to me. Also why are you using a table for your navigation? Read the link in my sig titled "Why Tables for Layout is Stupid?".
    Yeah, I know. I made the site quite a few years ago. Didn't know much then. I'll get around to taking the table out eventually. But for the update I needed the new pages in quickly, not a whole revamp of the site.

    Quote Originally Posted by SB65
    Sorry, could have been clearer - I get the scrollbar in IE7, not in FF3.
    Ah, ok. Well, that's better at least :P Less things to fix.


  •  

    Posting Permissions

    • You may not post new threads
    • You may not post replies
    • You may not post attachments
    • You may not edit your posts
    •