The Web Design Group

... Making the Web accessible to all.

Welcome Guest ( Log In | Register )

 
Reply to this topicStart new topic
> Kindly Review My website, I am a college student, and i made a personal website as a sort of res
jackz02
post Apr 1 2024, 04:51 PM
Post #1





Group: Members
Posts: 1
Joined: 1-April 24
Member No.: 29,154



https://github.com/JackZammit02/PersonalWebsite

It is my first time ever using HTML so please don't be too harsh.
User is offlinePM
Go to the top of the page
Toggle Multi-post QuotingQuote Post
Christian J
post Apr 2 2024, 12:11 PM
Post #2


.
********

Group: WDG Moderators
Posts: 9,722
Joined: 10-August 06
Member No.: 7



I only checked the code of the home.html file:

Instead of the generic div.header and div#myTopNav, you might use the dedicated HEADER and NAV elements.

CODE
<i class="fa fa-fw fa-home"></i>

The above seems to be used by the Font-awesome library to create menu icons. The I element seems semantically unsuitable there, and all the CSS classes make it look a bit bloated. Instead you might use CSS background-images on the A elements for the icons, along with a little padding-left to create space.

It's safest to specify a CSS background-color with together with the text color. If say a CSS background-image doesn't load, you may otherwise end up with white text color on a white background.

Some of the CSS comments seem a bit redundant, basically saying the same as the CSS rules themselves. tongue.gif

Using px units for font size used to be problematic, at least before browsers added zoom and minimum font-size user settings. https://www.w3.org/TR/2024/CRD-css-values-3...bsolute-lengths says that absolute length units like px "are mainly useful when the output environment is known" (such as for print stylesheets), while
https://www.w3.org/TR/css-fonts-3/#font-size-prop says that "Use of percentage values, or values in ems, leads to more robust and cascadable style sheets." unsure.gif
User is offlinePM
Go to the top of the page
Toggle Multi-post QuotingQuote Post

Reply to this topicStart new topic
2 User(s) are reading this topic (2 Guests and 0 Anonymous Users)
0 Members:

 



- Lo-Fi Version Time is now: 5th October 2024 - 10:57 AM