Skip to content

Conversation

@MizuRyzu
Copy link

No description provided.

vytvoril som novu komponentu empty pre csgov
Vytvoril som novu komponentu error code pre csgov
<div size="l" class="gov-spacer hydrated"></div>
<div variant="primary" size="m" type="solid" hoverable="" class="gov-button hydrated"><button class="element" type="button" tabindex="0" aria-label="{{ error_code_arial_label }}" aria-pressed="false" aria-disabled="false">{{ error_code_button_text }}</button>
</div>
</div>
Copy link
Member

Choose a reason for hiding this comment

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

Změnit indent ze 4 mezer na 2.

Copy link
Member

@petrillek petrillek left a comment

Choose a reason for hiding this comment

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

Už se blížíme. :)

<div size="l" class="gov-spacer hydrated"></div>
<div variant="primary" size="m" type="solid" hoverable="" class="gov-button hydrated"><button class="element" type="button" tabindex="0" aria-label="{{ error_code_arial_label }}" aria-pressed="false" aria-disabled="false">{{ error_code_button_text }}</button>
<div size="l" class="gov-spacer"></div>
<div variant="primary" size="m" type="solid" hoverable="" class="gov-button"><button class="element" type="button" tabindex="0" aria-label="{{ error_code_arial_label }}" aria-pressed="false" aria-disabled="false">{{ error_code_button_text }}</button>
Copy link
Member

Choose a reason for hiding this comment

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

error_code_arial_label – není to ariaL, ale aria. Navíc nahoře v komentáři to máš jako error_code_button_aria_label (to je správně).

Copy link
Member

Choose a reason for hiding this comment

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

Ještě ti tam chybí ten error_code_button_aria_label. ;)

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