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

Defect Hunting Fundamental NGX 0.22 [2] #3420

Closed
salarenko opened this issue Sep 28, 2020 · 7 comments
Closed

Defect Hunting Fundamental NGX 0.22 [2] #3420

salarenko opened this issue Sep 28, 2020 · 7 comments
Assignees
Labels
core Core library specific issues Defect Hunting denoland

Comments

@salarenko
Copy link
Contributor

salarenko commented Sep 28, 2020

Defect Hunting before v0.22.0 release.

@salarenko salarenko added core Core library specific issues Defect Hunting 0.22.0 labels Sep 28, 2020
@JKMarkowski
Copy link
Contributor

JKMarkowski commented Sep 28, 2020

FIREFOX

Inna: I checked FF and Chrome, both have 32px height:
Screen Shot 2020-09-29 at 4 49 33 PM
Screen Shot 2020-09-29 at 4 49 50 PM

@katekozlowska
Copy link
Contributor

katekozlowska commented Sep 28, 2020

CHROME

  • Popover placement: Inna PR
    deploy-preview-3400--fundamental-ngx netlify app_ (1)

  • Tokens are not centered - Inna: this issue is related to styles. I opened an issue there Token in compact mode is not vertically aligned fundamental-styles#1731
    deploy-preview-3400--fundamental-ngx netlify app_

  • Avatar on shellbar is broken - Inna PR
    deploy-preview-3400--fundamental-ngx netlify app_ (2)

  • Table - adding new item: close button is missaligned - not reproducible after Buttons PR was merged
    deploy-preview-3400--fundamental-ngx netlify app_ (3)

  • Carrets (arrows) are not centered (see seconds example) - not reproducible after Buttons PR was merged
    deploy-preview-3400--fundamental-ngx netlify app_ (4)

@salarenko
Copy link
Contributor Author

salarenko commented Sep 28, 2020

@katekozlowska
Copy link
Contributor

Several issues reported under FIrefox are also applicable to Chrome, please check fixes under both browsers.

@stefanoScalzo
Copy link
Contributor

Screen Shot 2020-09-28 at 10 24 48 AM
Table toolbar issues

@InnaAtanasova
Copy link
Contributor

Screen Shot 2020-09-28 at 10 24 48 AM
Table toolbar issues

@stefanoScalzo
If you meant buttons issues in the table, they've been fixed with the Buttons PR. Here's how it's in master now:
Screen Shot 2020-09-29 at 4 24 24 PM

If you meant other issues, please specify

@InnaAtanasova
Copy link
Contributor

All issues have been addressed and the PRs have been merged. Awesome job everyone!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
core Core library specific issues Defect Hunting denoland
Projects
None yet
Development

No branches or pull requests

6 participants