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

Removes Skeleton from the Learn page and replaces the flex implementa… #781

Merged
merged 3 commits into from
May 21, 2019

Conversation

Pjb518
Copy link
Contributor

@Pjb518 Pjb518 commented May 20, 2019

…tion in the footer with Tachyon styles. Adds custom CSS code which affect the global font size, as well as letter spacing and font weight for the section and page headings sitewide.

Pertains to this issue: #780

…tion in the footer with Tachyon styles. Adds custom CSS code which affect the global font size, as well as letter spacing and font weight for the section and page headings sitewide.
@skade skade requested review from yoshuawuyts and skade and removed request for yoshuawuyts May 20, 2019 15:53
@skade
Copy link
Contributor

skade commented May 20, 2019

Thanks for the patch!

There's two tiny issues on 1024x768

Screenshot_2019-05-20 Learn - Rust programming language(1)
(iPad resolution if you use the Firefox mobile view simulator).

The horizontal lines and the buttons are not aligned.

Screenshot_2019-05-20 Learn - Rust programming language(2)

The buttons and the text are not aligned here.

src/main.rs Outdated Show resolved Hide resolved
@Pjb518
Copy link
Contributor Author

Pjb518 commented May 20, 2019

Looks like the alignment issue exists in Chrome as well, and is also present in the current live site.

Firefox Live site:
Screenshot from 2019-05-20 18-10-01

Chrome Live site:
Screenshot from 2019-05-20 18-11-38

I'll submit a fix shortly.

…t issues with misaligned buttons in the get started with rust section. Adds several new Tachyons classes to fix alignment issues between links and labels in the Grow with Rust section.
…r match the original layout. Adds some temporary classes to prevent Skeleton classes interfering with Tachyon only styles due to shared class names.
@skade
Copy link
Contributor

skade commented May 20, 2019

@Pjb518 This looks quite good now. Am I overlooking anything or is this ready to merge?

@pietroalbini
Copy link
Member

An existing inconsistency of that page is the padding the first and last columns have.

Screenshot

2019-05-20--20-56-16

In all the other Tachyons pages that extra margin is not present. This is also visible in any other page's footer:

Screenshot

2019-05-20--21-01-59

@skade
Copy link
Contributor

skade commented May 21, 2019

Merging this to allow people to depend on the header and footer fixes. Outstanding issues tracked as #786

@skade skade merged commit d2af8e2 into rust-lang:master May 21, 2019
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.

3 participants