Skip to content

Fix UI and UX experience issues in main branch - #86

Merged
marcelo-m7 merged 4 commits into
mainfrom
copilot/fix-ui-ux-experience-errors
Oct 27, 2025
Merged

marcelo-m7 merged 4 commits into
mainfrom
copilot/fix-ui-ux-experience-errors

Conversation

Copilot AI commented Oct 27, 2025 •

Copy link
Copy Markdown

UI/UX Fixes for FACODI Portal

Issues Fixed:

  • Fix double /pt/pt/ in course URLs (routing issue) ✅
  • Fix misaligned text labels in course details sidebar (word-breaking issue) ✅
  • Fix translation keys mapping in i18n files ✅
  • Improve menu navigation (Cursos link routing) ✅
  • Fix console warnings about missing Supabase configuration ✅
  • Add proper error handling for Supabase connection ✅
  • Improve responsive design for mobile devices ✅
  • Add proper spacing and alignment for UI components ✅
  • Fix typography and text rendering issues ✅
  • Add proper language switcher functionality (requires additional backend work)

Security:

  • Fix vite vulnerability (updated from 7.0.6 to 7.1.11)

Changes Made:

1. Fixed URL Routing

  • Added explicit url frontmatter to course, UC, and topic pages
  • Fixed double /pt/pt/ issue - now URLs are clean: /pt/courses/LESTI/2024-2025/

2. Fixed CSS and Typography

  • Fixed word-breaking in definition lists (labels now display correctly: "Instituição", "Escola", "ECTS", "Duração", "Idioma")
  • Added responsive spacing improvements
  • Added smooth hover effects on cards
  • Improved heading and list spacing
  • Better badge and button styling
  • Mobile-responsive navigation improvements

3. Fixed Menu Navigation

  • Updated all language menu files to use correct language-specific paths
  • "Cursos" link now correctly points to /pt/ instead of /
  • Applied same fix to EN, ES, and FR menus

4. Improved Error Handling

  • Changed Supabase warnings from console.warn to console.info in development mode
  • Added development mode detection (localhost/127.0.0.1)
  • More user-friendly messages when Supabase is not configured
  • Graceful degradation when dynamic content is unavailable

5. Build Configuration

  • Updated .gitignore to exclude generated files (hugo_stats.json, tmp-screenshots/)

Screenshots:

Homepage with fixed navigation:
Homepage

Course page with fixed CSS and routing:
Course Page

Testing:

  • Build succeeds without errors
  • All pages render correctly
  • Navigation works properly
  • Console messages are appropriate
  • Responsive design verified
  • Typography renders correctly
Original prompt

there are many erros in the branch main in the ui and ux experience, like desilined comopents, translation mecanis withh error, not implement auto use github api for searching for repository etc.. names incorects, pages router, formins sending menu, etic etc. audit, test, create e start a plen for the fixings


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

Copilot AI self-assigned this Oct 27, 2025
Copilot AI and others added 2 commits October 27, 2025 16:59
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 marcelo-m7 left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@copilot please continue your work

Co-authored-by: marcelo-m7 <117441129+marcelo-m7@users.noreply.github.com>

Copilot AI commented Oct 27, 2025

Copy link
Copy Markdown
Author

@copilot please continue your work

Continued and completed the UI/UX fixes!

Fixed in commit 61cb00f:

✅ Routing Issues: Fixed double /pt/pt/ URLs - now clean paths like /pt/courses/LESTI/2024-2025/

✅ 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 (/pt/, /en/, etc.)

✅ 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:

Homepage with fixed navigation:
Homepage

Course page with all fixes applied:
Course Page

Copilot AI changed the title [WIP] Fix UI and UX experience issues in main branch Fix UI and UX experience issues in main branch Oct 27, 2025
Copilot AI requested a review from marcelo-m7 October 27, 2025 20:28
@marcelo-m7
marcelo-m7 marked this pull request as ready for review October 27, 2025 20:30
@marcelo-m7
marcelo-m7 merged commit 5432eb1 into main Oct 27, 2025
2 checks passed
@marcelo-m7
marcelo-m7 deleted the copilot/fix-ui-ux-experience-errors branch October 27, 2025 20:30

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 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".

Comment thread config/_default/hugo.toml
Comment on lines 45 to +46
[permalinks]
courses = "/courses/:sections[1:]/"
course = "/courses/:slug/"

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

P1 Badge 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 👍 / 👎.

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