I would appreciate some constructive criticism about my portfolio page. The projects are not added, because I need to find most of them.
The first page is quite bland, so I might animate the icons. I am just unsure of the best way to do without a performance drop.
Also, the logic in the Professional and Retro buttons is such that you could freeze your device, if you hit them many many times without hitting Default.
Kudos on using a <button> for your hamburger icon, I can actually get to it with the keyboard. I see you even have the aria-expanded attribute on it, but when I expand the menu that attribute stays at āfalseā. Also, after expanding the menu, the actual menu items are above the button in the tab index so I have to shift-Tab to get to them. I think most people would be expecting to Tab forward into them after expanding the menu.
When you mouseover the social media icons at the bottom they slide to show their name. I think it would be nice if they did this when they got keyboard focus as well. Also, at larger text sizes the icons do not slide over enough to expose the entire name.
Iām not seeing any text associated with the svg icons below your name. If these icons represent important information you want everyone to āseeā then Iād add a text alternative.
Iām not seeing an <h1> header on this page, only three <h4>s. Add an h1 and then make sure not to skip levels as you work your way down. Semantically, Iād add a header for each of your links in the menu. You could always visually hide them if you donāt want them on the page.
And believe it or not but my color contrast checker is saying that the white text on blue buttons does not have enough contrast to be accessible.
Your page handles text size increases well except that the projects could be better. At 200% increase, when my browser is wide they still look good. But as I narrow, the project divs get real thin, to the point where they can only display one letter per line. Then, when I finally reach your media query break point they switch to a single column and everything looks good again. My suggestion would be to base your break point on āemā instead of āpxā. Also, the buttons are breaking out of their containers.
Also, when I have my browser window narrow (so that the projects are in one column) and mouseover a project, it expands and the sides are cut off by the browser window and the bottom is under the project beneath it. This doesnāt take away from the functionality but I think some people might consider this a mistake.
There are a lot of ways to add text alternatives for these svg icons. A good page that describes the options is
But if you are just using them for decorations/animations and will be adding content that lists all of your skills then you probably donāt need to worry about this.
As far as I can see, I have updated my portfolio to include your earlier suggestions/reviews. Would you mind giving it a once-over for something I completely missed?
Hi, @Sky020! My feedback is going to focus on UX/accessibility/design. Some of the accessibility items are things I still need to do to my own portfolio/projects.
The Professional/Retro/Default options are very fun and downright hypnotic. I donāt know if I should lean back in my chair or lean forward into my screen! Both? //crash
I do recommend making that top nav black and the menu option text (About, Projects, Contact) white for accessibility (contrast) and simplicity (the grey/green donāt show up anywhere else on the site).
Recommend adding more descriptive text alternatives for all images/SVG graphics, so screen-readers can know whatās going on when they get to those images.
When the user hovers over the project āviewā buttons, the text turns a color that does not meet contrast standards for accessibility. I recommend changing this.
I donāt think the slide over text explanations of the freeCodeCamp/Linkedin, etc icons are 100% necessary, for two reasons. A) Theyāre fairly well-known icons, especially Facebook and Linkedin. B) Itās inconsistent UX-wise, because the row of icons in the beginning have some (comparatively) far more niche/obscure ones in their midst and donāt have any text explaining what they are. If you want to keep the slide over text explanations, I recommend adding hover over text to the first row of icons to maintain consistency (in terms of helping users understand what the icons represent).
Recommend changing the LinkedIn icon to a circular one, since the other 3 icons are circles. Again, just a consistency thing.
Recommend using the Courier New font somewhere else besides your name at the top of the page. It would create more visual continuity for the user if, as they scroll down the page, the project titles were also Courier New.
Consider using a specific type scale for font sizes. This is seriously magical (Golden Ratio is a personal favorite). https://type-scale.com/
Thank you, for this. I have changed the colours for better contrast. Also added tooltips for the svgs in the beginning.
I wanted to have the LinkedIn icon to be circular, but it looks much worse like that . Also, I prefer having the current font the way it is - the name does not fit, if it is the same font.
I like the type scaling, but Golden Ratio is too high, in my opinion. The Augmented Fourth is as high as I will go.
Since you are at a very high level technical level. I will be very hard onto your design choices. Since you seem to need it.
1: When i switch between retro then proffesional the color paterns remain the same
2: people wonāt know that there is more down bellow your not guiding the usser at all
3: there are to much icons they look like a collection of bounching fridge magnets or something a child would play with
4: i get a cookie warning if i look at your source. I am from europe and you are suppose to warn me
5: again not very user friendly when i scrolled down i had no idea i could click onto the buttons and can think they where just icons. A site design must help you reach your goal. Your goal is to guide your future employers through the site and read the information about you. So they can contact you but, they cant do that if they have no idea that something is clickable. You might work in IT but most people from HR and recrtuiters donāt
6: what is the use of the buttons that are jumping? does it mean you mastered them? if so why canāt i read your skill level
Itās a nice site from a techical perspective but, it does seem to lack almost every thing that would make a site work well for itās user design wise. There is no clear concept behind the site. The user is not guided at all. The site color scheme seems for children and is way to busy.
@Sky020I viewed your website on my cell phone so this will be from a mobile perspective. This is strictly my opinion on what you can do to improve the website.
In mobile view the icons do not line up properly. The react icon is almost touching the git icon.
I think you should remove the spinning icon effect from the two icons since the others do not spin. This is not consistent. How about you have them all fade in slowly? Fading in from opacity 0 to 1 is light on resources.
The project tiles should have images of what they look like not screenshots of the code itself.
I suggest removing the small Freecodecamp test suite window from the projects before taking the screenshots. I would not even show the green hamburger menu collapsed.
I do not think that the three buttons ādefaultā, āprofessionalā, or āretroā buttons and the background effects are necessary.
Use a real domain name like āShaunHamilton . comā if it is available.
I am still unsure of what to do, in terms of animations. I guess fading would be more natural.
Some of my projects do not have images (they are just scripts), and I did not want to leave them blank. Nothing I can do about this, except replace them with UI-based projects.
I noticed them, but you are expected to leave the test suite on for validation.
I thought the site too plain and boring, without.
I am not in a position to pay for a domain, but I have been thinking about changing my GitHub username (if possible).