-
-
Notifications
You must be signed in to change notification settings - Fork 287
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: Open Blogs Link in a New Tab for Better UX #903
Conversation
👷 Deploy request for v2-digitomize pending review.Visit the deploys page to approve it
|
✅ Deploy Preview for digitomize ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
WalkthroughThe Changes
Assessment against linked issues
Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media? TipsChatThere are 3 ways to chat with CodeRabbit:
Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (invoked as PR comments)
Additionally, you can add CodeRabbit Configration File (
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Actionable comments posted: 0
Review Details
Configuration used: CodeRabbit UI
Review profile: CHILL
Files selected for processing (2)
- client/src/components/NewNavbar.jsx (2 hunks)
- client/src/components/globals/Footer.jsx (1 hunks)
Additional comments not posted (4)
client/src/components/NewNavbar.jsx (3)
79-80
: Refactored className attribute for cleaner code.
94-106
: Updated "Blogs" link to open in a new tab for better UX.
120-127
: Adjusted conditional rendering logic for the "Admin" link based on user role and pathname.client/src/components/globals/Footer.jsx (1)
126-134
: Added "Blogs" link to the footer under the "Connect" section.
It doesn't seem to work for navbar |
okay, i ll look into it |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Actionable comments posted: 0
Review Details
Configuration used: CodeRabbit UI
Review profile: CHILL
Files selected for processing (1)
- client/src/components/NewNavbar.jsx (2 hunks)
Files skipped from review as they are similar to previous changes (1)
- client/src/components/NewNavbar.jsx
@pranshugupta54 kindly check it out now. |
https://deploy-preview-903--digitomize.netlify.app/ Why is it still opening in same? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Actionable comments posted: 0
Review Details
Configuration used: CodeRabbit UI
Review profile: CHILL
Files selected for processing (1)
- client/src/components/globals/Navbar/NewNavbar.jsx (3 hunks)
Additional comments not posted (1)
client/src/components/globals/Navbar/NewNavbar.jsx (1)
154-180
: Ensure consistency in link handling for "Blogs".The implementation for opening the "Blogs" link in a new tab using
target="_blank"
andrel="noopener noreferrer"
is correct and follows best practices for external links. This change should enhance user experience by not disrupting the current navigation state.
@pranshugupta54 I have changed the global navbar. it should be working right now. |
It opens a new page but goes to home page. (https://deploy-preview-903--digitomize.netlify.app) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Actionable comments posted: 0
Review Details
Configuration used: CodeRabbit UI
Review profile: CHILL
Files selected for processing (1)
- client/src/components/globals/Navbar/NewNavbar.jsx (3 hunks)
Files skipped from review as they are similar to previous changes (1)
- client/src/components/globals/Navbar/NewNavbar.jsx
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Actionable comments posted: 0
Review Details
Configuration used: CodeRabbit UI
Review profile: CHILL
Files selected for processing (1)
- client/src/components/NewNavbar.jsx (2 hunks)
Files skipped from review as they are similar to previous changes (1)
- client/src/components/NewNavbar.jsx
@pranshugupta54 Done. |
🚀 Congratulations on the successful PR merge! 🌟 Thank you for your dedication! @all-contributors please add @Vimall03 for code @holopin-bot @Vimall03 c-ch |
Congratulations @Vimall03, you just earned a holobyte! Here it is: https://holopin.io/holobyte/clwdzojws09970cl4bigvcapd This badge can only be claimed by you, so make sure that your GitHub account is linked to your Holopin account. You can manage those preferences here: https://holopin.io/account. |
I couldn't determine any contributions to add, did you specify any contributions? I've put up a pull request to add @Vimall03! 🎉 |
Hey @pranshugupta54 Extremely sorry for for the inconvenience. Thankyou! |
Pull Request Details
Description
Opens Blogs Link in a New Tab for Better UX & Added Blogs Link in Footer under Connect Section
Fixes
Fixes #893
Type of PR
Summary
This pull request adds functionality to open the Blogs link in a new tab for improved user experience. Additionally, it includes the addition of the Blogs link in the footer under the Connect section.
Additional Notes
Checklist
npm run lint:fix
andnpm run format:fix
.Summary by CodeRabbit
New Features
Refactor
className
attributes in navigation elements.