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

Color primitives and aliases in rustdoc #29264

Merged
merged 2 commits into from Oct 29, 2015
Merged

Conversation

mdinger
Copy link
Contributor

@mdinger mdinger commented Oct 24, 2015

This is an alternative to #29240 which fixes #15307 by adding colors to primitives and aliases instead of underlining.

Try to keep the discussion in #29240 for now though so it can be kept track of.

A sample rendering is here

@rust-highfive
Copy link
Collaborator

r? @alexcrichton

(rust_highfive has picked a reviewer for you, use r? to override)

@alexcrichton
Copy link
Member

r? @steveklabnik

@steveklabnik
Copy link
Member

I think that I like this change better, but I'm going to hold off until after the branch today to do it. Sneaking something like this in right before the release doesn't seem like the right thing to do.

@mdinger
Copy link
Contributor Author

mdinger commented Oct 26, 2015

@steveklabnik Any opinion on making all the types a single color instead of varying them? Also, should I add the colored underline on hover? Both are easy to do.

@defuz
Copy link
Contributor

defuz commented Oct 26, 2015

Just examples of how underline on hover looks like:

img1

And:

img2

@steveklabnik
Copy link
Member

@bors: r+ rollup

The branch has happened, so let's merge this and see what people think. I expect it landing on nightly will let people give even more/better feedback.

@bors
Copy link
Contributor

bors commented Oct 28, 2015

📌 Commit 4cb013e has been approved by steveklabnik

@steveklabnik
Copy link
Member

(ie, i think this is a decent solution, but can imagine changing / reverting before 1.6 beta happens)

@mdinger
Copy link
Contributor Author

mdinger commented Oct 28, 2015

Ok cool. I'll file an issue to revisit this later then.

@mdinger
Copy link
Contributor Author

mdinger commented Oct 28, 2015

Oy, I never pushed the change which made macro links green. Here I fixed it.

r? @steveklabnik

@steveklabnik
Copy link
Member

@bors: r+ rollup

@bors
Copy link
Contributor

bors commented Oct 28, 2015

📌 Commit 11c7bdd has been approved by steveklabnik

steveklabnik added a commit to steveklabnik/rust that referenced this pull request Oct 29, 2015
This is an alternative to rust-lang#29240 which fixes rust-lang#15307 by adding colors to primitives and aliases instead of underlining.

Try to keep the discussion in rust-lang#29240 for now though so it can be kept track of.

A sample rendering is [here](http://mdinger.github.io/rust_std_colored/std/index.html)
bors added a commit that referenced this pull request Oct 29, 2015
@bors bors merged commit 11c7bdd into rust-lang:master Oct 29, 2015
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.

Std docs links are not all highlighted
6 participants