Feedback on church website

It's a very simple website admittedly, still getting there in designing stuff. As I'm not getting paid for this job, I'm not looking to do anything over the top, but if there are opportunities to add some flare to the website without spending a significant amount of dev time, I'm super open to it! Any feedback would be much appreciated, thank you! https://sweet-creponne-9e8302.netlify.app/ Aside, big ask: if anyone would like to skim through my repo to point out better ways to code that would also be appreciated. https://github.com/v11ncent/Christ-Church-Sedalia
24 Replies
croganm
croganmβ€’11mo ago
I know this doesn't provide any helpful feedback but I do want to say it looks great! For not getting paid, you've been very generous with the product you're providing πŸ˜‚. Only thing I noticed is the image is not staying within the border. Not sure if this is on purpose but I think it'd look better if it matched the other images πŸ˜‚
croganm
croganmβ€’11mo ago
No description
vince
vinceβ€’11mo ago
Hey thanks man, that means a lot! πŸ™‚ And the borders were intentional, I can try to mess around with that design as it is a bit out of place
croganm
croganmβ€’11mo ago
Yea absolutely! And don't get me wrong, it really is a fantastic design, you should be proud. It was just the one tiny thing I noticed that felt out of place only because it was the one place it was used and no where else on the site. Doing something unique there is not a bad idea, I just try not to mess with the border radius used throughout the rest of the site
vince
vinceβ€’11mo ago
For sure man thank you πŸ™‚
Senra
Senraβ€’11mo ago
This might not be constructive feedback, but The heading background psuedo-elements you make are really subtle yet eye catching. They take the simple heading to a unique look and feel. I think you can introduce these elements in other routes like get in touch, donate etc.
No description
vince
vinceβ€’11mo ago
Good point, those pages need some flare. Thank you!
Senra
Senraβ€’11mo ago
Is that called a flare huh
vince
vinceβ€’11mo ago
Nah just what I call it haha
Senra
Senraβ€’11mo ago
Well that makes sense. I will adopt that terminology 🫑
vince
vinceβ€’11mo ago
Thanks Hope! This feedback is really great πŸ™‚ I never thought about the arrow being misleading like that. I'll make those changes πŸ™‚ πŸ‘ Haha yea, I really have no idea what I'm doing in the business department and need all the help I can get! I'll make a thread there πŸ™‚ Definitely!
briancross
briancrossβ€’11mo ago
Looks great! I really like the font in the headings. The only inconsistency I noticed was the Donate and Events hero sections are left justified while all others are centred. Cool use of backgrounds!
vince
vinceβ€’11mo ago
Thank you! I did that on purpose cause I thought it looked a bit weird with them being left justified with the content that they have
briancross
briancrossβ€’11mo ago
Ah, I figured that would be intentional! I was looking through your code. Plausible analytics.. interesting. I gotta check that out. I really want to start using Astro as well. I’m assuming you can import client side scripts rather than having it all in a script tag?
vince
vinceβ€’11mo ago
It's really good and $9/month for the starter plan So I'm not 100% on how Astro handles client scripts. I believe when you write JS in a script tag inside a .astro component, then it loads it inline. If you import it though, it should load a script rather than inline-ing it
briancross
briancrossβ€’11mo ago
Yeah that is a good deal.
vince
vinceβ€’11mo ago
It's based off page views too. Only up to 10,000 monthly page views with the lowest-tier plan but that is way more than I currently get so far so I don't need to worry about that haha
croganm
croganmβ€’11mo ago
It actually will still process and bundle everything in the script tag unless you put a is:inline or any other property other than src πŸ™‚ From Astro:
Astro’s default bundling behavior will be disabled in some situations. In particular, adding type="module" or any attribute other than src to a <script> tag will cause Astro to treat the tag as if it had an is:inline directive. The same will be true when the script is written in a JSX expression.
ghostmonkey
ghostmonkeyβ€’11mo ago
great work, you have come a long way in your design skills! If I would make any changes, I would experiment with the button hover transition. At .3s it feels almost laggy, but I really like it changed to .15s, feels much more snappy and in the hero, i might also try adding a max-width on the pargraph text (it looks good at 55ch and 70ch on a desktop, so somewhere in that range), while also putting more space below it to separate the buttons a bit. and maybe a bit more space between the buttons so they aren't right on top of each other
ghostmonkey
ghostmonkeyβ€’11mo ago
No description
ghostmonkey
ghostmonkeyβ€’11mo ago
and also looking at it, maybe remove 'come' from the title, and just say 'Worship Jesus With Us'
vince
vinceβ€’11mo ago
Thanks ghostmonkey! Great suggestions. I'll add them today πŸ™‚πŸ‘ Made your changes. Also added Astro's ViewTransitions. I'm loving it
ghostmonkey
ghostmonkeyβ€’11mo ago
it's good work, you should be proud of it definitely make sure it's in your portfolio
vince
vinceβ€’11mo ago
Thank you, definitely will add it. Waiting for the client to wrap it up and going to write a blog/case study on it
Want results from more Discord servers?
Add your server
More Posts