Fix author name visibility in dark mode with professional horizontal…#643
Fix author name visibility in dark mode with professional horizontal…#643Kiretychowdary wants to merge 1 commit into
Conversation
… layout - Redesigned author layout from vertical circular to horizontal card style - Fixed author name visibility issues in dark mode - Added circular avatar on left with name beside it - Implemented social links row below author name - Added delightful hover effects and animations - Ensured perfect contrast in both light and dark themes - Made layout responsive for mobile devices - Updated styling across blog carousel and main blog pages Fixes author name display issues for better UX in Hacktoberfest 2025
|
@Kiretychowdary is attempting to deploy a commit to the recode Team on Vercel. A member of the Team first needs to authorize it. |
|
Thank you for submitting your pull request! 🙌 We'll review it as soon as possible. The estimated time for response is 5–8 hrs. In the meantime, please provide all necessary screenshots and make sure you run - npm build run , command and provide a screenshot, a video recording, or an image of the update you made below, which helps speed up the review and assignment. If you have questions, reach out to LinkedIn. Your contributions are highly appreciated!😊 Note: I maintain the repo issue every day twice at 8:00 AM IST and 9:00 PM IST. If your PR goes stale for more than one day, you can tag and comment on this same issue by tagging @sanjay-kv. We are here to help you on this journey of open source. Consistent 20 contributions are eligible for sponsorship 💰 🎁 check our list of amazing people we sponsored so far: GitHub Sponsorship. ✨ 📚Your perks for contribution to this community 👇🏻
If there are any specific instructions or feedback regarding your PR, we'll provide them here. Thanks again for your contribution! 😊 |
There was a problem hiding this comment.
Pull Request Overview
This PR redesigns the author display layout from a vertical circular design to a horizontal card-style layout, with a primary focus on fixing author name visibility issues in dark mode.
- Redesigned author layout to use horizontal card style with avatar on left and name beside it
- Fixed author name visibility problems in dark mode by adding explicit color overrides and text shadows
- Updated package.json dependencies from Docusaurus 3.8.1 to 3.9.1
Reviewed Changes
Copilot reviewed 4 out of 4 changed files in this pull request and generated 3 comments.
| File | Description |
|---|---|
| src/pages/blogs/blogs-new.css | Restructured author component from vertical to horizontal layout with enhanced styling |
| src/css/custom.css | Added comprehensive author styling overrides for both light and dark themes with animations |
| src/components/blogCarousel/blogCarousel.css | Applied horizontal author layout styling to blog carousel components |
| package.json | Updated Docusaurus dependencies from 3.8.1 to 3.9.1 |
Comments suppressed due to low confidence (1)
src/css/custom.css:1
- Excessive use of !important declarations makes CSS difficult to maintain and override. Consider using more specific selectors instead of relying on !important for all properties.
/* You can override the default Infima variables here. */
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
| } | ||
|
|
||
| /* ================= SIMPLE CIRCULAR AUTHOR DESIGN ================= */ | ||
| /* � Clean & Perfect Circular Author Layout */ |
There was a problem hiding this comment.
There's a rendering issue with the emoji character. Replace '�' with an appropriate emoji like '🎯' or remove it entirely.
| /* � Clean & Perfect Circular Author Layout */ | |
| /* 🎯 Clean & Perfect Circular Author Layout */ |
| display: flex !important; | ||
| flex-direction: row !important; | ||
| align-items: flex-start !important; | ||
| justify-content: flex-start !important; | ||
| width: auto !important; | ||
| height: auto !important; | ||
| border-radius: 16px !important; | ||
| background: linear-gradient(135deg, #f8fafc 0%, #e2e8f0 100%) !important; | ||
| border: 2px solid #e2e8f0 !important; | ||
| padding: 20px !important; | ||
| box-shadow: | ||
| 0 4px 15px rgba(0, 0, 0, 0.08), | ||
| 0 2px 8px rgba(0, 0, 0, 0.04) !important; | ||
| transition: all 0.3s ease !important; | ||
| text-align: left !important; | ||
| position: relative !important; | ||
| max-width: 320px !important; | ||
| min-height: 120px !important; |
There was a problem hiding this comment.
This CSS rule for .avatar is duplicated earlier in the file (lines 1536-1553) with conflicting styles. The horizontal layout defined here conflicts with the circular layout defined earlier, creating unpredictable behavior.
| background: linear-gradient(135deg, #4f46e5 0%, #7c3aed 50%, #ec4899 100%); | ||
| box-shadow: | ||
| 0 6px 20px rgba(79, 70, 229, 0.4), | ||
| 0 3px 10px rgba(124, 58, 237, 0.3); | ||
| border: 2px solid rgba(99, 102, 241, 0.3); |
There was a problem hiding this comment.
This dark theme styling for .author-avatar conflicts with the earlier dark theme rule at lines 273-277 which has different background and border styles. This creates inconsistent styling behavior.
| background: linear-gradient(135deg, #4f46e5 0%, #7c3aed 50%, #ec4899 100%); | |
| box-shadow: | |
| 0 6px 20px rgba(79, 70, 229, 0.4), | |
| 0 3px 10px rgba(124, 58, 237, 0.3); | |
| border: 2px solid rgba(99, 102, 241, 0.3); | |
| /* Consolidated dark theme styles for consistency */ | |
| background: linear-gradient(135deg, #4f46e5 0%, #7c3aed 50%, #ec4899 100%); | |
| box-shadow: | |
| 0 6px 20px rgba(79, 70, 229, 0.4), | |
| 0 3px 10px rgba(124, 58, 237, 0.3); | |
| border: 3px solid #8b5cf6; |
|
Hello sir can you check it once the changes i have made it and it will look good for the users to observe the author name |
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
|
There is some change in the UI required. #530 you can refer this repo blog for code base. |
|
sorry sir I will check it make it proper way sorry for any conflicts but I will make it better to you with good UI |
|
yeah sir as I have made the change because while I am visiting the web page I can see that the author name is looking small so for the better purpose I have made those changes if you need to change it before one I can do it but for better looking I have been done it sir , If you need it I can create a new pull request sir ? |
|
@sanjay-kv Design review. |
|
@Kiretychowdary , lets stick the design we had previously. |
There was a problem hiding this comment.
Pull Request Overview
Copilot reviewed 4 out of 4 changed files in this pull request and generated 3 comments.
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
| [data-theme='dark'] .avatar { | ||
| position: relative; | ||
| border: 3px solid transparent; | ||
| border-radius: 16px; | ||
| background: linear-gradient(135deg, rgba(15, 23, 42, 0.95) 0%, rgba(30, 41, 59, 0.9) 100%); | ||
| backdrop-filter: blur(12px); | ||
| box-shadow: | ||
| 0 8px 32px rgba(79, 70, 229, 0.15), | ||
| 0 4px 16px rgba(124, 58, 237, 0.1), | ||
| inset 0 1px 0 rgba(255, 255, 255, 0.1); | ||
| padding: 16px; | ||
| transition: all 0.4s cubic-bezier(0.175, 0.885, 0.32, 1.275); | ||
| overflow: hidden; | ||
| } |
There was a problem hiding this comment.
This dark theme avatar styling is later overridden by the horizontal layout styles starting at line 1867. The conflicting styles create unnecessary complexity and could lead to unexpected behavior.
| .avatar { | ||
| display: flex !important; | ||
| flex-direction: column !important; | ||
| align-items: center !important; | ||
| justify-content: center !important; | ||
| width: 140px !important; | ||
| height: 140px !important; | ||
| border-radius: 50% !important; | ||
| background: linear-gradient(135deg, #f8fafc 0%, #e2e8f0 100%) !important; | ||
| border: 3px solid #6366f1 !important; | ||
| padding: 16px !important; | ||
| box-shadow: | ||
| 0 6px 20px rgba(99, 102, 241, 0.25), | ||
| 0 3px 10px rgba(139, 92, 246, 0.15) !important; | ||
| transition: all 0.3s ease !important; | ||
| text-align: center !important; | ||
| position: relative !important; | ||
| } | ||
|
|
||
| [data-theme='dark'] .avatar { | ||
| background: linear-gradient(135deg, #1e293b 0%, #334155 100%) !important; | ||
| border-color: #8b5cf6 !important; | ||
| box-shadow: | ||
| 0 6px 20px rgba(139, 92, 246, 0.3), | ||
| 0 3px 10px rgba(124, 58, 237, 0.2) !important; | ||
| } |
There was a problem hiding this comment.
This circular avatar layout is completely overridden by the horizontal layout starting at line 1867. Having multiple conflicting style definitions for the same class creates maintainability issues and makes the code harder to understand.
| .avatar { | |
| display: flex !important; | |
| flex-direction: column !important; | |
| align-items: center !important; | |
| justify-content: center !important; | |
| width: 140px !important; | |
| height: 140px !important; | |
| border-radius: 50% !important; | |
| background: linear-gradient(135deg, #f8fafc 0%, #e2e8f0 100%) !important; | |
| border: 3px solid #6366f1 !important; | |
| padding: 16px !important; | |
| box-shadow: | |
| 0 6px 20px rgba(99, 102, 241, 0.25), | |
| 0 3px 10px rgba(139, 92, 246, 0.15) !important; | |
| transition: all 0.3s ease !important; | |
| text-align: center !important; | |
| position: relative !important; | |
| } | |
| [data-theme='dark'] .avatar { | |
| background: linear-gradient(135deg, #1e293b 0%, #334155 100%) !important; | |
| border-color: #8b5cf6 !important; | |
| box-shadow: | |
| 0 6px 20px rgba(139, 92, 246, 0.3), | |
| 0 3px 10px rgba(124, 58, 237, 0.2) !important; | |
| } |
| display: flex !important; | ||
| flex-direction: row !important; | ||
| align-items: flex-start !important; | ||
| justify-content: flex-start !important; | ||
| width: auto !important; | ||
| height: auto !important; | ||
| border-radius: 16px !important; | ||
| background: linear-gradient(135deg, #f8fafc 0%, #e2e8f0 100%) !important; | ||
| border: 2px solid #e2e8f0 !important; | ||
| padding: 20px !important; | ||
| box-shadow: | ||
| 0 4px 15px rgba(0, 0, 0, 0.08), | ||
| 0 2px 8px rgba(0, 0, 0, 0.04) !important; | ||
| transition: all 0.3s ease !important; | ||
| text-align: left !important; | ||
| position: relative !important; | ||
| max-width: 320px !important; | ||
| min-height: 120px !important; |
There was a problem hiding this comment.
Excessive use of !important declarations makes CSS harder to maintain and override. Consider using more specific selectors instead of relying on !important for every property.
| display: flex !important; | |
| flex-direction: row !important; | |
| align-items: flex-start !important; | |
| justify-content: flex-start !important; | |
| width: auto !important; | |
| height: auto !important; | |
| border-radius: 16px !important; | |
| background: linear-gradient(135deg, #f8fafc 0%, #e2e8f0 100%) !important; | |
| border: 2px solid #e2e8f0 !important; | |
| padding: 20px !important; | |
| box-shadow: | |
| 0 4px 15px rgba(0, 0, 0, 0.08), | |
| 0 2px 8px rgba(0, 0, 0, 0.04) !important; | |
| transition: all 0.3s ease !important; | |
| text-align: left !important; | |
| position: relative !important; | |
| max-width: 320px !important; | |
| min-height: 120px !important; | |
| display: flex; | |
| flex-direction: row; | |
| align-items: flex-start; | |
| justify-content: flex-start; | |
| width: auto; | |
| height: auto; | |
| border-radius: 16px; | |
| background: linear-gradient(135deg, #f8fafc 0%, #e2e8f0 100%); | |
| border: 2px solid #e2e8f0; | |
| padding: 20px; | |
| box-shadow: | |
| 0 4px 15px rgba(0, 0, 0, 0.08), | |
| 0 2px 8px rgba(0, 0, 0, 0.04); | |
| transition: all 0.3s ease; | |
| text-align: left; | |
| position: relative; | |
| max-width: 320px; | |
| min-height: 120px; |
|
Sir @Adez017 but while i am visiting the below UI design was displaying in light mode
|
|
if this pull request is done perfectly I am very happy to do more changes to your web application that look wonderful to the users please trust me I will make look awesome and everything that helps to you sirs @Adez017 @sanjay-kv |
thats Correct @Kiretychowdary , but we want it to be fixed. the UI which I and @sanjay-kv has mentioned looks good with the overall website and thats why we are pointing to fix the existing one . if you are okay with it , try to fix that one . |
|
Ok sir I will do it |
|
Fixed here |
iitzIrFan
left a comment
There was a problem hiding this comment.
My suggestion would be to stick to the previous design and resolve this, issue ASAP ! @Kiretychowdary
| "@docusaurus/preset-classic": "3.9.1", | ||
| "@docusaurus/theme-classic": "3.9.1", | ||
| "@docusaurus/theme-mermaid": "3.9.1", | ||
| "@docusaurus/theme-search-algolia": "3.9.1", |
| "@docusaurus/types": "^3.8.1", | ||
| "@docusaurus/module-type-aliases": "3.9.1", | ||
| "@docusaurus/tsconfig": "3.9.1", | ||
| "@docusaurus/types": "3.9.1", |














… layout