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

feat: Improved UI of FAQ page to match website theme #1003

Closed
wants to merge 0 commits into from

Conversation

TBorundia
Copy link
Contributor

@TBorundia TBorundia commented Jul 6, 2024

Title and Issue number

Title : Improved UI of FAQ page to match website theme

Issue No. : #986

Code Stack :

Close #986

Description

Improved UI to match website theme.
Made the page responsive too.

Video/Screenshots (mandatory)

travelWeb_faq.mp4
travelWeb_faq_responsive.mp4

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

Additional context:

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

  • I am contributing under GSSOC'24
  • I am contributing under VSOC'24

Copy link

netlify bot commented Jul 6, 2024

Deploy Preview for taupe-cendol-f7e2bb ready!

Name Link
🔨 Latest commit 2517b56
🔍 Latest deploy log https://app.netlify.com/sites/taupe-cendol-f7e2bb/deploys/6693740ae981f800092ead5b
😎 Deploy Preview https://deploy-preview-1003--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.

@TBorundia
Copy link
Contributor Author

TBorundia commented Jul 7, 2024

@apu52 @itsekta , please review this PR.
Thank You!

@TBorundia TBorundia closed this Jul 13, 2024
@apu52
Copy link
Owner

apu52 commented Jul 13, 2024

@TBorundia is there any problem that u have closed this PR as I was reviewing this PR & suddenly it is closed

@TBorundia TBorundia reopened this Jul 13, 2024
@TBorundia
Copy link
Contributor Author

TBorundia commented Jul 13, 2024

@apu52 Sir, I am experiencing an issue with the background image .
I will reopen the task and would appreciate your feedback on whether it appears the same for you.
If the issue persists, I will open a new PR else you can merge this PR.
Thank you.

@apu52 apu52 changed the title Improved UI of FAQ page to match website theme feat: Improved UI of FAQ page to match website theme Jul 14, 2024
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.

@TBorundia exactly as u have suggested , the website is really looking messy in your PR . So I would like to better suggest u to submit a new PR making a new branch & push all of your changes there in your forked repo & please close this PR

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
wontfix This will not be worked on
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Feature Request]:UI of FAQ page to match website theme
2 participants