282-feat: Standardize links designation

[Wystov]

What type of PR is this? (select all that apply)

• 🍕 Feature

Description

Important

02.09.24 PO approved links design change in accordance to WCAG.

• changed color and added underline of textVariant of CustomLink according to WCAG
• reviewed all links on website and changed them to LinkCustom where applicable.
• LinkCustom opens in new window now if 'external' prop passed to it. also if it's an textVariant of link, it will be marked with symbol indicating external link.
• removed font size from textVariant of link, so it can inherit it from text where it's used.
• changed about.data format to tsx, so we can use directly jsx from it and not just setInnerHtml from string as it worked before.
• set default icons for variants, but they can be still replaced by manually passing icon prop to component
• added 'custom' variant that has no default styling and relies on styles and classes passed as props

Related Tickets & Documents

• Related Issue Standardize Designation of Internal and External Links Site-Wide #282
• Closes Standardize Designation of Internal and External Links Site-Wide #282

Screenshots, Recordings

internal text link

external text link

Added/updated tests?

• 👌 Yes

[optional] What gif best describes this PR or how it makes you feel?

Summary by CodeRabbit

Summary by CodeRabbit

• New Features

  • Introduced a new blue color variable for consistent styling across the application.
  • Enhanced handling of external links with the new LinkCustom component, improving user experience and accessibility.

• Bug Fixes

  • Simplified link rendering logic by removing unnecessary icon props, ensuring a cleaner user interface.

• Chores

  • Streamlined imports and removed unused components to improve code maintainability.

[ansivgit]

There's not our styles for text links O-O

[Wystov]

There's not our styles for text links O-O

@ansivgit luckily i dont have any disabilities but i still barely can distinguish links from other text in "our styles". let alone the fact that they're completely failing WCAG criteria (i left a link in pr description, please check it out)

[SpaNb4]

run visual now

[SpaNb4]

run visual now

[SpaNb4]

run visual now

[SpaNb4]

run visual now

[SpaNb4]

run visual now

[SpaNb4]

run visual now

[SpaNb4]

run visual now

[SpaNb4]

run visual now

[SpaNb4]

run visual now

[SpaNb4]

run visual now

[SpaNb4]

run visual now

[github-actions]

Lighthouse Report:

• Performance: 76 / 100
• Accessibility: 98 / 100
• Best Practices: 100 / 100
• SEO: 100 / 100

View detailed report

[github-actions]

Lighthouse Report:

• Performance: 96 / 100
• Accessibility: 98 / 100
• Best Practices: 100 / 100
• SEO: 100 / 100

View detailed report