Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

story(sbb-timetable-row): review/refactor #1291

Closed
9 tasks
osminaz opened this issue Jul 14, 2022 · 0 comments
Closed
9 tasks

story(sbb-timetable-row): review/refactor #1291

osminaz opened this issue Jul 14, 2022 · 0 comments
Assignees

Comments

@osminaz
Copy link
Contributor

osminaz commented Jul 14, 2022

Feature Description

Feature Description

We want to review/refactor the sbb-timetable-row implementation.

Figma Specs
Timetable results Specs

Timetable-row Specs

Timetable-row Skeleton

Card Specs

References

Design Spec

Technical Spec

  • should be a native button -> Emits an event onClick ( -> wrapper: timetable-row-button)

  • assumption: timetable-row-button and timetable-button can be summarize in one component

  • provide aria attribute for the button: aria-label, aria-haspopup, aria-expanded and aria-controls

  • aria-expanded should toggle from false to true onClick

  • set the role to "presentation" for the components button-wrapper and the role="row", role="rowHeader" and role="gridcell" for the inner Elements

  • provide a state for loading

  • We want to use these two components as slots:

    • sbb-card-badge,
    • sbb-pearl-chain
  • We want to provide flat Props, which will be orientated to the visual specs:

    • Up:
      • pictogram: string
      • transportNumber: string // could be an icon or a rendered string?
      • direction: string <-- truncate logic
    • Middle:
      • walkTimeBefore?: string, // footPath
      • walkTimeAfter?: string, // footPath
      • leftTime: string, // departureTime - tbd: delay?
      • rightTime: string, // arrivalTime - tbd: delay?
    • bottom:
      • plattform: string tbd: variants?
      • occupancyFirstClass?: "unknown" | "low" | "medium" | "high"
      • occupancySecondClass?: "unknown" | "low" | "medium" | "high"
      • travelHints?: Array
      • duration: string,
      • warning: string

Questions:
slots or props?
Do we want to provide a list of possible icons?

<sbb-timetable-row>
  <sbb-card-badge/>
  <sbb-pearl-chain/>
</sbb-timetable-row>

Markup:

<sbb-timetable-row-button>
  <sbb-card>
      <slot name="cardBadge">
      ...
      ...
      <slot name="pearlChain">
      ...
      ...
   </sbb-card>
</sbb-timetable-row-button>

--> should resolve in (#1299)

Definition of Done

  • Component is implemented
  • Tests are implemented
  • E2E Tests are implemented
  • Storybook stories are implemented
  • Navigation via keyboard is tested
  • Screen reader output is tested
  • High-contrast is tested
  • Remaining accessibility is tested
  • UX approved
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

No branches or pull requests

3 participants