SAP / fundamental-styles

SAP Design System component library for building SAP user interfaces with any web technology.
https://sap.github.io/fundamental-styles
Apache License 2.0
185 stars 60 forks source link

Before release Defect Hunting + Bug fixes #3327

Closed InnaAtanasova closed 2 years ago

InnaAtanasova commented 2 years ago

Browsers for testing:

Themes for testing:

RTL and LTR testing too

Components for testing:

mikerodonnell89 commented 2 years ago
Screen Shot 2022-04-15 at 10 53 20 AM Screen Shot 2022-05-07 at 14 33 52 Screen Shot 2022-05-07 at 14 34 21
platon-rov commented 2 years ago
image

Current:

image

Specs:

image

Specs:

image
droshev commented 2 years ago
droshev commented 2 years ago
N1XUS commented 2 years ago
InnaAtanasova commented 2 years ago

The icon is not visible. (Morning Horizon)

Screen Shot 2022-05-09 at 2 19 46 PM

(HCB Horizon)

Screen Shot 2022-05-09 at 2 19 37 PM
InnaAtanasova commented 2 years ago
InnaAtanasova commented 2 years ago

Fundamental-styles:

Screen Shot 2022-05-09 at 2 32 02 PM Screen Shot 2022-05-09 at 2 31 26 PM

Design specs:

Screen Shot 2022-05-09 at 2 31 49 PM
InnaAtanasova commented 2 years ago

Design specs:

Screen Shot 2022-05-09 at 3 15 34 PM
InnaAtanasova commented 2 years ago

Fund-styles:

Screen Shot 2022-05-09 at 3 54 58 PM

Specs:

Screen Shot 2022-05-09 at 3 57 42 PM
InnaAtanasova commented 2 years ago

The focus in active state should be white:

Screen Shot 2022-05-09 at 4 45 52 PM
InnaAtanasova commented 2 years ago
InnaAtanasova commented 2 years ago
Screen Shot 2022-05-09 at 6 58 12 PM
InnaAtanasova commented 2 years ago
platon-rov commented 2 years ago
platon-rov commented 2 years ago

Create a separate issue for future testing and integration with ngx

moved to https://github.com/SAP/fundamental-styles/issues/3495

platon-rov commented 2 years ago

WON'T FIX Note: Shouldn't have a border.

platon-rov commented 2 years ago

To be honest i didn't find specifications for this component, in the wiki i only found: Icon Tab Bar, Multi-Instance TabContainer, Wizard. In case Tabs component were renamed to Icon Tab Bar we should also deprecate Tabs.

Inna: IconTabBar should be used instead of Tabs. We kept Tabs because apps were using it, but it's no longer supported and you won't find specs for it.

platon-rov commented 2 years ago
image
platon-rov commented 2 years ago
platon-rov commented 2 years ago
image
g-cheishvili commented 2 years ago

image

N1XUS commented 2 years ago
g-cheishvili commented 2 years ago

Current: image

Specs:

image

g-cheishvili commented 2 years ago
g-cheishvili commented 2 years ago

image

g-cheishvili commented 2 years ago

image

platon-rov commented 2 years ago
image
N1XUS commented 2 years ago

@platon-rov: not yet implemented, https://github.com/SAP/fundamental-styles/issues/3247

platon-rov commented 2 years ago

image

mikerodonnell89 commented 2 years ago
mikerodonnell89 commented 2 years ago

@platon-rov: not yet implemented, https://github.com/SAP/fundamental-styles/issues/3247

g-cheishvili commented 2 years ago

image

g-cheishvili commented 2 years ago
g-cheishvili commented 2 years ago
g-cheishvili commented 2 years ago

image

g-cheishvili commented 2 years ago
platon-rov commented 2 years ago