brianacampos/termproject

TermProject

by Briana Campos

termproject's Issues

Technical Issue 1: Javascript linked to text instead of button

Hi Briana,
I love what you are doing with the "Who we are" section on the homepage, and I wanted to let you know that when you click the "more" it is working as intended, but when you click "less" nothing happens. You can see that the function is working when you click text content rather than the button:

TermProject/index.html

Line 284 in 64e89b9

clickAny.textContent = "";

Try copying the code from where you made the action of showing the content when you click the button rather than the text for a more intuitive user experience.

Content Issue 1: Suggest Replace or Crop Stretched Image

Hi Briana,

Overall I LOVE the imagery on your site; you get a great sense of the vibe in the Bay Area. I would suggest you either crop or replace the image below that serves as the background for the "Restaurants" header:

TermProject/index.html

Lines 125 to 128 in 64e89b9

	src="images/koisushi.jpg"
	alt="restaurant"
	title="restaurant"
	style="width: 100%"

I am sure there are much yummier, more enticing restaurant images you can find to draw the user into exploring more! I do love the overlays with the images and button, though, so definitely suggest you keep that.

Content Issue 2: Suggest Style Change Where Restaurants Overlaps with Logo

Hi Briana,

First, I love the fading effect you have on your header in the "Restaurants" page. However, it makes it a bit difficult to read once it's faded. I'd suggest either moving it to another line or maybe changing the color to one with high contrast with your blue logo, like maybe black or red.

Here's the CSS I'm referring to:

TermProject/content.css

Lines 72 to 77 in 64e89b9

	span.bold{
	font-weight: 700;
	color: #758BFD;
	text-transform: uppercase;
	font-style: italic;
	font-size: 4rem;

And here's the location in your header:

TermProject/restaurants.html

Line 34 in 64e89b9

<h1 class="logo">

Great work!

Technical Issue 2: Feedback Form Overlaps Header - CSS Needed

Hi Briana,

If you take a look below you'll find that the feedback form is overlapping with the header, making it so the user cannot fill the first two fields. I searched your CSS files and it seems you did not add CSS for the class you assigned here, "feedback-form":

TermProject/contact-us.html

Line 78 in 64e89b9

<section class="feedback-form">

I'm guessing you just forgot to style this element and you can use similar styling to the other types throughout the website, but I wanted to bring your attention to this bug so that you'll be able to get feedback on your site!

Issues and suggestions

Goodthings: Very rich content and beautiful grid design! I really like the visuals!

Technical suggestion:
1.for the about us div, the show less button doesn't work. consider add:
type="button", in the button div, otherwise its default is submit
2.for the Featured Content image show room, the dots below the image doesn't work.
dont use inline operator. consider replace the very long function plusSlides, showSlides with img.src=""

  <a class="prev" onclick="plusSlides(-1)">&gt;</a>
      <a class="next" onclick="plusSlides(1)">&lt;</a>

Content suggestion:

1. Maybe can make the website more interactive, such as clicking or hovering on the image and a different image of the same place show up.
2. for the outdoor section, another heading showed up and overlapped.