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.

Sign up for GitHub

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

Jump to bottom

Browsers Preview #4

Open
vv-monsalve opened this issue Jan 11, 2022 · 3 comments
Open

Browsers Preview #4

vv-monsalve opened this issue Jan 11, 2022 · 3 comments

Comments

@vv-monsalve
Copy link
Contributor

vv-monsalve commented Jan 11, 2022 •
edited
Loading

Sierra Safari 11
Desktop_OS_X_High_Sierra_safari_11 1_
Windows 10 Chrome_71
Desktop_Windows_10_chrome_71 0_
Windows 10 Edge_18
Desktop_Windows_10_edge_18 0_
@vv-monsalve
Copy link
Contributor Author

Browser Images from files at commit bd568cc

OS X High Sierra Safari_11
Desktop_OS_X_High_Sierra_safari_11 1_
Windows 10 Chrome_71
Desktop_Windows_10_chrome_71 0_
Windows 10 Edge_18
Desktop_Windows_10_edge_18 0_

@vv-monsalve
Copy link
Contributor Author

Latest images after mastering process changes. Apart from what we've talked about the revision on particular cases (e.g. ¿ spacing) I think the horizontal regression (caused for a spacing/kerning improvement) should be confirmed by @davelab6 or @m4rc1e.

OS X High Sierra Safari_11
Desktop_OS_X_High_Sierra_safari_11 1_
Windows 10 Chrome_71
Desktop_Windows_10_chrome_71 0_
Windows 10 Edge_18
Desktop_Windows_10_edge_18 0_

@davelab6
Copy link
Member

It's a mix of shorter and longer lines, and generally if shorter I think it's good, but since the spacing already had some issues, I think no care needs to be taken t ok retain original spacing in this case. Make it much better please and let the chips fall where they may

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@davelab6 @vv-monsalve