looking for costructive criticism on my site!!!

[moeghost]

ive been working on my website, moeghost, for around 2 or 3 years  now. ive been coding since 2020 so i already had some experience, but i took a year or 2 long break at one point so theres still quite a bit that im learning and want to learn and improve on. i only recently learned how to do iframes, turns out its really not all that complicated!!!

anyways, be truly honest about what u think!! unless ur just being mean i guess...im looking for a "this is great" or "this could use this" type of answer rather than "this sucks lol." feel free to even snoop around my code :p

i can honestly say that this is my proudest creation and im pretty happy with how everything looks at the moment, i just would like to know how i could improve it and what others think!! 

[larvapuppy]

I wrote a really long reply but the forum signed me out while I was writing it and it got deleted 

I'll summarize I guess:

• I really like the cute pink quilt look and how everything matches!! also Chii!
• Your site suffers from scrollbarception; all the little scrolling sections within scrolling sections are quite annoying to navigate, plus the fact that your huge main content scrolls AND the entire page scrolls is a little unnecessary (also the large section scrolls horizontally, do overflow-x:hidden to prevent this). Pick and choose what you want to scroll and especially let the smaller sections on your sidebar just be the length that they are
• Also if you are going to have all these scrollbars you should style them cute and pink to match the rest of the site 
• The Chii on your landing page is super cute but I don't find landing pages super necessary, people who hate javascript and gifs are likely just going to click away when they see them anyway, same with people on mobile if your site looks fucked. If you're worried about accessibility, implement a gif pauser or reduce the # of fast gifs (but I don't personally find your site that gif heavy), remove autoplay (tho it doesn't seem to be working for me), and consider doing some small css edits to make things more mobile friendly
• Nav bar: The links to unfinished pages are not necessary, but if you must keep them: the font makes it hard to tell that they have strikes through them and linking directly to the 404 page instead of their future urls is a little silly

Sorry for being nitpicky and obviously take what I say with a grain of salt bc it's opinionated! I think your site is really great already but a few small changes could bring it up to 100% 

[arcus]

It's really cute! I really like the patterns and colours you chose, it's a lot like a 2000s site.

I don't know how much help I can be since I keep JavaScript and GIFs disabled though. Looking at some of your code:

You don't need to add semicolons at the end of single line style tags.

Code

<center style="margin-top:20px;">

Should be

Code

<center style="margin-top:20px">

<center> tags are old, and it's suggested to use CSS to centre things instead these days. You can keep using them if you want though.

There's </br> tags but no

Code

<br>

tags on the splash page.

div id is used for stuff that you use once and is usually linked to. You can use class instead of id and it should be ok.

None of those are a big deal. You did a really good job with it.

The Chii on your landing page is super cute but I don't find landing pages super necessary, people who hate javascript and gifs are likely just going to click away when they see them anyway, same with people on mobile if your site looks fucked.

It's not always a matter of opinion with JavaScript. Many have it disabled for accessibility, security, or other safety reasons. Anyone that doesn't have it enabled can't tell if a site is intentionally empty or not without a warning or by checking the source code. Most JavaScript avoiders are fine with enabling JavaScript as long as they know who coded it and what it's used for. I personally like the warning, but think it could be more descriptive and note if the site is still usable with it disabled or not. <noscript> tags can be used to hide the warning from anyone with JavaScript enabled.

If you're worried about accessibility, implement a gif pauser or reduce the # of fast gifs (but I don't personally find your site that gif heavy)

There's flaws with those solutions that make them not ideal for preventing photosensitive epilepsy. The only safe way involves converting GIFs into videos, and no one wants to do that. A warning on the splash page is nice because anyone that still wants to view the site can manually disable GIF animations in their browser, and enable them again when they leave.

[moeghost]

• Your site suffers from scrollbarception; all the little scrolling sections within scrolling sections are quite annoying to navigate, plus the fact that your huge main content scrolls AND the entire page scrolls is a little unnecessary (also the large section scrolls horizontally, do overflow-x:hidden to prevent this).
• Also if you are going to have all these scrollbars you should style them cute and pink to match the rest of the site 
• The Chii on your landing page is super cute but I don't find landing pages super necessary...remove autoplay (tho it doesn't seem to be working for me), and consider doing some small css edits to make things more mobile friendly
• Nav bar: The links to unfinished pages are not necessary, but if you must keep them: the font makes it hard to tell that they have strikes through them and linking directly to the 404 page instead of their future urls is a little silly

Sorry for being nitpicky and obviously take what I say with a grain of salt bc it's opinionated! I think your site is really great already but a few small changes could bring it up to 100% 

thank you!!! i can definitely agree with pretty much everything you said, i kind of prefer nitpicky responses that mention the little things!! i like that you say it has a "quilt look," i always thought the same!!

i can very much agree that scrolling on the homepage is a bit annoying, i think i will just make the whole main div longer so its just the whole page and little sections that use scrollbars. i keep the scrollbars at 0px on the homepage tho just bc they can take up a bit of room. once i change this ill probably add the older looking scrollbar thats on some of my other pages

for the sideways scrollbar that you mentioned, i dont have this? maybe it depends on screen size?

honestly the landing page was just an excuse for me to create another page for fun haha, a lot of my fav sites have them and i just like having a little mascot to welcome you in!! and kind of just shortening what the other reply said, i think its good just to have a little warning anyways

as for the autoplay music, it appears my player itself needs fixing....the play/pause button doesnt work so i just let it autoplay for now until i fix it, i think most browsers have a mute site option so this isnt something ive really prioritized. will definitely look into how i can make it more accessible for mobile users tho!!

the unfinished links are just to remind me what i want to add and to show people what i have planned. i can agree about the font, its really cute but even the links without a del tag look as if they have a dash through them...this is also something i will be tweaking a bit!!

It's really cute! I really like the patterns and colours you chose, it's a lot like a 2000s site.

<center> tags are old, and it's suggested to use CSS to centre things instead these days. You can keep using them if you want though.

There's </br> tags but no [/list]

Code

<br>

tags on the splash page.

thank you!! ive done my best to give it an early 2000s blog/sozai site feel. im glad you like!!

i use center tags a lot of the time because sometimes "margin:auto" or "text-align:center" dont work for whatever reason. center tags kind of always work. i mostly just use this for images, tho.

i use br tags mainly just to separate lines of text and sometimes just to add a little gap. ive never found the starting tag to be necessary, just the closing tag does the trick for what i use it for.

div id is used for stuff that you use once and is usually linked to. You can use class instead of id and it should be ok.

None of those are a big deal. You did a really good job with it.

i use both interchangeably, i only sometimes have issues w this. most of the time i do stick with class for divs and id for specific text or images. it gets the job done either way tho usually

thank you both for the honesty and feedback!! i will definitely be taking all of this into consideration!! 

[lakes]

It's a very impressive & cute site, however I find pastel pink text on a lighter pink background hard to read. Maybe you can add a bit more contrast between the text & background. Maybe have the background for the text be cream-colored, so it doesn't blend in too much, while preserving that cutesy aesthetic.