Fix UI and UX experience issues in main branch - #86
Conversation
Co-authored-by: marcelo-m7 <117441129+marcelo-m7@users.noreply.github.com>
Co-authored-by: marcelo-m7 <117441129+marcelo-m7@users.noreply.github.com>
marcelo-m7
left a comment
There was a problem hiding this comment.
@copilot please continue your work
Co-authored-by: marcelo-m7 <117441129+marcelo-m7@users.noreply.github.com>
Continued and completed the UI/UX fixes! Fixed in commit ✅ Routing Issues: Fixed double ✅ CSS/Typography: Fixed word-breaking in sidebar labels - now displays correctly: "Instituição", "Escola", "ECTS", "Duração", "Idioma" ✅ Navigation: Fixed menu links to use correct language paths ( ✅ Error Handling: Improved console messages - changed warnings to info messages in dev mode ✅ Responsive Design: Added better spacing, hover effects, and mobile-friendly styles Screenshots: |
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
| [permalinks] | ||
| courses = "/courses/:sections[1:]/" | ||
| course = "/courses/:slug/" |
There was a problem hiding this comment.
Preserve course code when generating permalinks
Changing the permalink rule from courses = "/courses/:sections[1:]/" to course = "/courses/:slug/" removes the course code from generated URLs. For content bundles under content/pt/courses/<code>/<plan>/index.md Hugo now emits /courses/<plan>/ by default, so two different courses that share a plan version will collide and later builds will override or shadow pages unless every file defines a manual url. The previous rule automatically included both nested sections (LESTI/2024-2025) and avoided clashes. Consider keeping the hierarchical sections in the permalink pattern or adding the course code to the template to prevent duplicate URLs when new courses are added.
Useful? React with 👍 / 👎.


UI/UX Fixes for FACODI Portal
Issues Fixed:
/pt/pt/in course URLs (routing issue) ✅Security:
Changes Made:
1. Fixed URL Routing
urlfrontmatter to course, UC, and topic pages/pt/pt/issue - now URLs are clean:/pt/courses/LESTI/2024-2025/2. Fixed CSS and Typography
3. Fixed Menu Navigation
/pt/instead of/4. Improved Error Handling
console.warntoconsole.infoin development mode5. Build Configuration
.gitignoreto exclude generated files (hugo_stats.json,tmp-screenshots/)Screenshots:
Homepage with fixed navigation:

Course page with fixed CSS and routing:

Testing:
Original prompt
💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.