Skip to content

Updates Ui navbar #85

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

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

AdarshProjects
Copy link

@AdarshProjects AdarshProjects commented Feb 10, 2025

What does this PR do

This pull request, updates the UI of the navbar . It includes a following changes -

  • Updated styling for a cleaner look.
  • Restructured some elements for better alignment.
  • Create a user friendly navbar.

Previous Navbar
Before keploy

Upates Navbar
After keploy

Closes keploy/keploy#2534

@AdarshProjects
Copy link
Author

@Sonichigo Could you please approve the workflow for my PR?

@Sonichigo Sonichigo self-requested a review February 13, 2025 11:03
@Hermione2408
Copy link
Member

@AdarshProjects please sign off DCO

@Hermione2408
Copy link
Member

Hermione2408 commented Mar 6, 2025

Screenshot 2025-03-06 at 4 13 38 PM

@AdarshProjects The current navbar implementation does not align with the intended design from the main landing page. The styling, hover effect, button styles,shadow and icon placements need to match the reference navbar. Please update the design to ensure consistency across the blog and main landing page.

@AdarshProjects
Copy link
Author

AdarshProjects commented Mar 11, 2025

Hi @Hermione2408,

video_github.mp4

I apologize for my previous PR not meeting the required standards. I've now updated the PR with all the necessary changes. Please have a look!

I have:

  • Applied all hover effects as requested
  • Updated styling and button placements for consistency
  • Ensured alignment with the main landing page

@Hermione2408
Copy link
Member

Hi @Hermione2408,

<video Uploading Engineering _ Keploy Blog and 1 more page - Personal - Microsoft_ Edge 2025-03-12 00-15-53 (1).mp4…/>

I accidentally pushed my latest changes to the closed PR instead of opening a new one. Since the updates include all the requested fixes, could you kindly reopen the PR so it can be reviewed and merged?

I have: -Applied all hover effects as requested -Updated styling and button placements for consistency

  • Ensured alignment with the main landing page

can you tell me which PR you are talking about?

@AdarshProjects
Copy link
Author

@Hermione2408
Sorry, I thought you had closed my PR, so I wrote that message. Apologies for the confusion. Anyway, I've applied the requested changes—please check this PR, I've updated it

@AdarshProjects AdarshProjects force-pushed the updating-ui-navbar branch 2 times, most recently from d78b0c2 to 18652ec Compare March 13, 2025 09:44
Signed-off-by: Adarsh Kumar <[email protected]>
@AdarshProjects
Copy link
Author

Hi @Hermione2408,
<video Uploading Engineering _ Keploy Blog and 1 more page - Personal - Microsoft_ Edge 2025-03-12 00-15-53 (1).mp4…/>
I accidentally pushed my latest changes to the closed PR instead of opening a new one. Since the updates include all the requested fixes, could you kindly reopen the PR so it can be reviewed and merged?
I have: -Applied all hover effects as requested -Updated styling and button placements for consistency

  • Ensured alignment with the main landing page

can you tell me which PR you are talking about?

Hi @Hermione2408 ,

I've updated the PR to fully match the requested requirements, including styling, hover effects, button placements, and alignment with the main landing page.

Additionally, I've resolved the previous DCO issues—apologies for any confusion earlier.

I'm 100% sure that these updates will fully address the issue. Whenever you get time, please review the PR. I truly appreciate your feedback and hope this will close the issue.

Thanks!

Copy link

⚠️ Oops! We currently support Unit Test Generation only for Go projects.
Stay tuned – support for more languages is coming soon! 🚀

2 similar comments
Copy link

⚠️ Oops! We currently support Unit Test Generation only for Go projects.
Stay tuned – support for more languages is coming soon! 🚀

Copy link

⚠️ Oops! We currently support Unit Test Generation only for Go projects.
Stay tuned – support for more languages is coming soon! 🚀

Copy link

Nice Pr 😄. We currently support Unit Test Generation only for Go projects.
Stay tuned – support for more languages is coming soon! 🚀

3 similar comments
Copy link

Nice Pr 😄. We currently support Unit Test Generation only for Go projects.
Stay tuned – support for more languages is coming soon! 🚀

Copy link

Nice Pr 😄. We currently support Unit Test Generation only for Go projects.
Stay tuned – support for more languages is coming soon! 🚀

Copy link

keploy bot commented Apr 15, 2025

Nice Pr 😄. We currently support Unit Test Generation only for Go projects.
Stay tuned – support for more languages is coming soon! 🚀

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.

[feature]: update ui of blog-website navbar
2 participants