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

fix: Logo rendering and fix styles #1637

Merged
merged 3 commits into from
Nov 10, 2024

Conversation

Zubaida0113
Copy link
Contributor

@Zubaida0113 Zubaida0113 commented Nov 10, 2024

Title and Issue number

Title : [Bug]: Redirecting links of Services tab is not working, Logo not rendering on website and Light/Dark Mode inconsistency between Book Now page and website.

Issue No. :#1629

Code Stack : Html, CSS, javascript

Close #1629

Description

Logo Rendering on Specific Pages: The logo should appear consistently across all pages, including the About Us, Top Deals, and Rate Us pages. The logo should render correctly regardless of the page to provide a uniform look across the website
Logo Mismatch on Certain Pages: Once clarified, either the orange background logo or the blue earth logo should be used consistently across all pages. The logo design should be identical on the Book Now, Plan a Trip, and Membership pages to avoid confusion and reinforce brand identity.
Services Tab Redirection: Each sub-service under the Services tab (such as Destinations, Trip Gallery, and Hotels and Resorts) should redirect to the correct page when clicked. The links should be updated to avoid "Not Found" errors, ensuring a smooth navigation experience for users.
Light/Dark Mode Synchronization: The Book Now page should synchronize its theme with the rest of the website. If the user is in dark mode on any page and navigates to Book Now, it should also display in dark mode, and vice versa. This synchronization should apply globally across all pages.
Navbar Button Design Consistency: The Login and View buttons should have a consistent design on all pages. The styling seen on the Home page should be applied across the About, Packages, and other sections, ensuring uniformity in button appearance across the website.

Video/Screenshots (mandatory)

Type of PR

  • Bug fix
  • Feature enhancement
  • Documentation update
  • Other (specify): _______________

Checklist:

  • I have performed a self-review of my code.
  • I have commented my code, particularly in hard-to-understand areas.
  • I have tested the changes thoroughly before submitting this pull request.
  • I have provided relevant issue numbers, screenshots, and videos after making the changes.
  • I have gone through the contributing.md file before contributing
  • I have Starred the Repository.

Additional context:

##Are you contributing under any Open-source programme?

  • I am contributing under GSSOC'24 Extended
  • I am contributing under Hacktoberfest'24

Copy link

Thank you for submitting your pull request! 🙌 We'll review it as soon as possible. In the meantime, If there are any specific instructions or feedback regarding your PR, we'll provide them here. Thanks again for your contribution! 😊

Copy link

netlify bot commented Nov 10, 2024

Deploy Preview for taupe-cendol-f7e2bb ready!

Name Link
🔨 Latest commit 3eed373
🔍 Latest deploy log https://app.netlify.com/sites/taupe-cendol-f7e2bb/deploys/6730a2cc2217310008d39f2f
😎 Deploy Preview https://deploy-preview-1637--taupe-cendol-f7e2bb.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.

@Zubaida0113 Zubaida0113 changed the title Logo rendering and fix styles fix: Logo rendering and fix styles Nov 10, 2024
@Zubaida0113
Copy link
Contributor Author

@apu52 Please review this PR as soon as possible.

Copy link
Owner

@apu52 apu52 left a comment

Choose a reason for hiding this comment

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

@Zubaida0113 good work , thanks for your valuable contribution

@apu52 apu52 merged commit f1542c1 into apu52:main Nov 10, 2024
6 of 7 checks passed
Copy link

🎉🎉 Thank you for your contribution! Your PR #1637 has been merged! 🎉🎉

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
2 participants