Skip to content

Touchups - #61

Merged
justinrubek merged 20 commits into
masterfrom
touchups
May 16, 2021
Merged

Touchups#61
justinrubek merged 20 commits into
masterfrom
touchups

Conversation

@byverone

Copy link
Copy Markdown
Collaborator

The home page (index.html) has been updated with the following:

  • Addition and removal of team members
  • Added new project
  • Changed "Schedule a Meeting" to "Free Consultation"

The About page (learnmore.html) has been updated with the following:

  • formating and style to match the home page

@byverone byverone added the enhancement New feature or request label May 13, 2021

@justinrubek justinrubek left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This looks pretty good. The navbar needs to be touched up though:

  • The Levity logo and text don't navigate from the about page to the home page, but rather they currently stay on the current page
  • Opening the about page currently opens in a new tab

This is how the main page's nav bar was made and it got copied over into the about page.
If they are on the about page then there is no way to return to the main site. Additionally, I would be annoyed by getting extra tabs in the process while we are on the same site. It makes sense for the external links so that they can stay on our site at least

@justinrubek

Copy link
Copy Markdown
Contributor

I'll be making these changes myself as they are simple and we want to get this deployed faster

@justinrubek justinrubek left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We are ready to deploy with the navbar fixes. This is all good

@justinrubek
justinrubek merged commit dafe0e6 into master May 16, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants