Skip to content

Fix website issues#9

Merged
Javier-varez merged 7 commits intomainfrom
fix-website-issues
Mar 23, 2026
Merged

Fix website issues#9
Javier-varez merged 7 commits intomainfrom
fix-website-issues

Conversation

@Javier-varez
Copy link
Copy Markdown
Member

  • Set text color to black. Page 4, point 2.
  • Use en-dash instead of hyphen for T4EQ.
  • Fix domain name reported by google by setting canonical domain. Page 1, point 1.
  • Add favicon with current logo. Page 1, point 3.
  • Bring fixes from theme (see related PR here).
  • Remove unused bio in teamgrid.

Reference for the changes:
https://drive.google.com/drive/u/0/folders/1CglyFHnGXePUpEiXOpcZ4AtT91wb0FP3

Copy link
Copy Markdown
Contributor

@preethipeie preethipeie left a comment

Choose a reason for hiding this comment

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

I don't see the font color change visually. However could see n-dash change take effect.

@Javier-varez
Copy link
Copy Markdown
Member Author

I don't see the font color change visually. However could see n-dash change take effect.

did you bring the changes from the theme? to me the colours are also not super noticeable, but I had a look with the current site side-by-side and I could definitely tell (also using the html inspector showed the right colours)

@Javier-varez
Copy link
Copy Markdown
Member Author

Also, thanks for the quick feedback!

@Javier-varez
Copy link
Copy Markdown
Member Author

I prepared some screenshots for reference. On the left is the current page. On the right the page with the fixes applied

Homepage

homepage homepage2

About page

about

en-dash detail

footer

favicon detail

favicon

@alice-poma
Copy link
Copy Markdown

Looks great!

@Javier-varez Javier-varez merged commit f8157dc into main Mar 23, 2026
1 check passed
@Javier-varez Javier-varez deleted the fix-website-issues branch March 23, 2026 17:11
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.

3 participants