Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fixes #830: Update index.html navigation by removing Blog and adding 'Start Free Trial' option #831

Closed
wants to merge 2 commits into from

Conversation

byteom
Copy link
Contributor

@byteom byteom commented Oct 24, 2024

Title of the Pull Request

  • Renamed the PR Title to reflect the changes: fix: Update navigation in index.html to remove blog option and add 'Start Free Trial'

Related Issue

Closes: #830

Description

In this pull request, I made updates to the navigation section of index.html. Specifically, I removed the "Blog" option from the navigation bar because it was already available in the "Links" section of the footer. Having the same link in both places caused misalignment in the navigation bar.

To maintain better alignment and balance, I replaced the "Blog" link in the navigation bar with a "Start Free Trial" option. This change not only improves the design but also enhances the user experience by making the free trial option more prominent.

Type of change

  • Code style update (formatting, alignment in navigation)
  • New feature (added 'Start Free Trial' to the navigation)

Checklist

  • My code follows the code style of this project.
  • I have followed the contribution guidelines.
  • I have performed a self-review of my own code.
  • I have ensured my changes don't generate any new warnings or errors.
  • I have updated the documentation (if necessary).
  • I have resolved all merge conflicts.

Copy link

vercel bot commented Oct 24, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
skillwise ✅ Ready (Inspect) Visit Preview 💬 Add feedback Oct 24, 2024 10:57pm

Copy link

netlify bot commented Oct 24, 2024

Deploy Preview for skillwise ready!

Name Link
🔨 Latest commit 48f0d72
🔍 Latest deploy log https://app.netlify.com/sites/skillwise/deploys/671c73b249f71a0008fc64fc
😎 Deploy Preview https://deploy-preview-831--skillwise.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@byteom
Copy link
Contributor Author

byteom commented Oct 25, 2024

/please accept this

@anuragverma108
Copy link
Collaborator

Conflicting files
style.css

Resolve Conflicts ?

Copy link

vercel bot commented Oct 26, 2024

Deployment failed with the following error:

Resource is limited - try again in 1 hour (more than 100, code: "api-deployments-free-per-day").

@byteom
Copy link
Contributor Author

byteom commented Oct 26, 2024

/conflict resolved

@PriyaGhosal
Copy link
Owner

@byteom
image
Just remove mentor from here ..... Blog is important button

@PriyaGhosal
Copy link
Owner

@byteom Are you working on it... as today is the last date of hacktoberfest 2024

@byteom byteom deleted the issue-830 branch November 2, 2024 14:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Navigation Bar Misalignment in Header Section
3 participants