Skip to content

Conversation

Copilot
Copy link
Contributor

@Copilot Copilot AI commented Aug 16, 2025

This PR reverts the changes made in PR #315 ("Replaced navibar and dashboard emojis with Professional Icons") and restores the original emoji-based icon system for navigation and dashboard components.

Changes Made

Removed Professional Icon Components

  • Deleted src/components/navbar/NavbarIcon.tsx - Component that mapped names to Lucide React icons
  • Deleted src/components/navbar/NavbarIconInjector.tsx - Component that dynamically injected professional icons into navbar elements

Restored Emoji-Based Navigation

Updated docusaurus.config.ts to use emojis directly with the nav-emoji CSS class instead of placeholder spans:

- html: '<span id="nav-docs"></span> Docs',
+ html: '<span class="nav-emoji">📚</span> Docs',

- html: '<span id="nav-dashboard"></span> Dashboard', 
+ html: '<span class="nav-emoji">📊</span> Dashboard',

Updated Dashboard Components

  • Dashboard sidebar navigation now uses emojis (📊 Home, 📺 Discuss, 🏅 Leaderboard, 💰 Giveaway)
  • Discussion category filters converted from Lucide icons to emojis (📢 Announcements, 💡 Ideas, ❓ Q&A, ⭐ Show & Tell, 💬 General)
  • Discussion tabs updated to use emoji indicators (📈 Trending, ❓ Unanswered)
  • Search functionality restored to use 🔍 emoji instead of Lucide Search icon

Cleaned Up Dependencies

  • Removed imports of NavbarIcon and NavbarIconInjector from all consuming components
  • Removed NavbarIconInjector usage from src/theme/Navbar/index.tsx
  • Preserved existing nav-emoji CSS classes that handle responsive emoji display

Emoji Mapping Applied

Component Original Professional Icon Restored Emoji
Docs/E-books Book 📚
Showcase Eye 👁️
Dashboard/Home LayoutDashboard 📊
Donate/Giveaway DollarSign 💰
Devfolio User 👤
Blogs Newspaper 📝
GitHub Github 🐙
Badges/Leaderboard BadgeCheck 🏅
Community Users 👥
Broadcast/Discuss Tv 📺
Podcast Mic 🎙️
Ideas/Behavioral Lightbulb 💡
Technical Puzzle 🧩
Announcements Megaphone 📢

Impact

This change restores the navigation bar and dashboard to their original state before PR #315, using the familiar emoji-based icon system. The existing responsive CSS for nav-emoji class ensures proper display across different screen sizes. All functionality remains unchanged - only the visual representation has been reverted from professional icons back to emojis.

Closes the issue requesting reversion of PR #315 changes.


✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

Copy link

vercel bot commented Aug 16, 2025

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Preview Comments Updated (UTC)
recode-website Ready Ready Preview Comment Aug 16, 2025 0:10am

@Copilot Copilot AI changed the title [WIP] Revert PR #315 and restore emoji-based nav bar and dashboard Revert PR #315: Restore emoji-based navigation and dashboard icons Aug 16, 2025
@Copilot Copilot AI requested a review from sanjay-kv August 16, 2025 12:13
Copilot finished work on behalf of sanjay-kv August 16, 2025 12:13
@sanjay-kv sanjay-kv closed this Aug 16, 2025
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.

2 participants