Reply to thread

Message

<blockquote data-quote="Sysode" data-source="post: 275340" data-attributes="member: 25453"><p>Like @[USER=221]Deion[/USER] said, it's like you've attempted to create something more advanced than you are capable of creating - it's fine, worth a try.</p><p></p><p>First thing I noticed is that you seemed to have found some tutorial on how to create 3D text using CSS - this is fine but it seriously does not suit your kind of design.</p><p>Also, there are some other things that need looking into:</p><ul>
<li data-xf-list-type="ul">The top-bar is the entire page width yet it only features a small logo in the middle? Either avoid going to with a top bar or add some content (navigation maybe) to it;</li>
<li data-xf-list-type="ul">The header image doesn't reach full width. You declared a width of what it must reach (1350px) which means it'll look decent on a screen resolution with a width of 1350.. Any smaller will cause scrolling by default and any larger (like mine) just looks bad.</li>
<li data-xf-list-type="ul">Consistency is key when designing, or so I think. You've used various hover-styles and padding for each section and it just makes your site look messy and not thought of. Try and stick to the same padding on each container and adding the same hover effects on &lt;div&gt;'s</li>
</ul><p>Also, to point out now as it's a good habit to get into, you're coding style seems a little outdated.. By this I mean:</p><ul>
<li data-xf-list-type="ul">Your code isn't valid at all. For example, if you take your image which you have used, you haven't included an alt tag. It should be like so[HTML]&lt;img src=&quot;../path/image.jpg&quot; alt=&quot;Your image alt&quot; /&gt;[/HTML]</li>
<li data-xf-list-type="ul">When using selectors for CSS, I notice that you use ID's (&lt;div id=&quot;&quot;&gt;) which is fine for smaller sites, but it's quicker and more efficient to use classes (&lt;div class=&quot;&quot;&gt;) &lt;- May be useful to get into the habit of that now;</li>
<li data-xf-list-type="ul">Also, this is just a personal preference but you don't indent your code at all? Maybe look into completing that as well.</li>
</ul><p>There are room for a few improvements, work on them and then maybe share your progress soon and gather some more feedback. Best of luck, hope this helps.</p></blockquote><p></p>

[QUOTE="Sysode, post: 275340, member: 25453"] Like @[USER=221]Deion[/USER] said, it's like you've attempted to create something more advanced than you are capable of creating - it's fine, worth a try. First thing I noticed is that you seemed to have found some tutorial on how to create 3D text using CSS - this is fine but it seriously does not suit your kind of design. Also, there are some other things that need looking into: [LIST] [*]The top-bar is the entire page width yet it only features a small logo in the middle? Either avoid going to with a top bar or add some content (navigation maybe) to it; [*]The header image doesn't reach full width. You declared a width of what it must reach (1350px) which means it'll look decent on a screen resolution with a width of 1350.. Any smaller will cause scrolling by default and any larger (like mine) just looks bad. [*]Consistency is key when designing, or so I think. You've used various hover-styles and padding for each section and it just makes your site look messy and not thought of. Try and stick to the same padding on each container and adding the same hover effects on <div>'s [/LIST] Also, to point out now as it's a good habit to get into, you're coding style seems a little outdated.. By this I mean: [LIST] [*]Your code isn't valid at all. For example, if you take your image which you have used, you haven't included an alt tag. It should be like so[HTML]<img src="../path/image.jpg" alt="Your image alt" />[/HTML] [*]When using selectors for CSS, I notice that you use ID's (<div id="">) which is fine for smaller sites, but it's quicker and more efficient to use classes (<div class="">) <- May be useful to get into the habit of that now; [*]Also, this is just a personal preference but you don't indent your code at all? Maybe look into completing that as well. [/LIST] There are room for a few improvements, work on them and then maybe share your progress soon and gather some more feedback. Best of luck, hope this helps. [/QUOTE]

Verification

Reply to thread

Connect with us

Newest members