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

doc: restore color for visited links #35108

Closed
wants to merge 1 commit into from
Closed

Conversation

Trott
Copy link
Member

@Trott Trott commented Sep 8, 2020

A previous change altered the rendering of visited links. This restores
it.

Checklist
  • make -j4 test (UNIX), or vcbuild test (Windows) passes
  • commit message follows commit guidelines

A previous change altered the rendering of visited links. This restores
it.
@nodejs-github-bot nodejs-github-bot added the doc Issues and PRs related to the documentations. label Sep 8, 2020
@Trott Trott mentioned this pull request Sep 8, 2020
2 tasks
@Trott
Copy link
Member Author

Trott commented Sep 8, 2020

@aduh95 @DerekNonGeneric @nodejs/website

Copy link
Contributor

@aduh95 aduh95 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@Trott
Copy link
Member Author

Trott commented Sep 8, 2020

This should fix this rendering:

image

@DerekNonGeneric
Copy link
Contributor

People don't want to know which links they have already visited?

Copy link
Contributor

@DerekNonGeneric DerekNonGeneric left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I see now why this can result in sub-par documentation browsing experience.

@silverwind
Copy link
Contributor

Could also use a slightly darker green for visited links. Sometimes it's a useful feature.

@Trott
Copy link
Member Author

Trott commented Sep 10, 2020

Collaborators, 👍 here to fast-track to get this landed in time for the 14.10.1 release.

Copy link

@willin willin left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@richardlau richardlau added commit-queue Add this label to land a pull request using GitHub Actions. fast-track PRs that do not need to wait for 48 hours to land. labels Sep 10, 2020
@github-actions github-actions bot removed the commit-queue Add this label to land a pull request using GitHub Actions. label Sep 10, 2020
@github-actions
Copy link
Contributor

Commit Queue failed
- Loading data for nodejs/node/pull/35108
✔  Done loading data for nodejs/node/pull/35108
----------------------------------- PR info ------------------------------------
Title      doc: restore color for visited links (#35108)
Author     Rich Trott  (@Trott)
Branch     Trott:green -> nodejs:master
Labels     doc, fast-track
Commits    1
 - doc: restore color for visited links
Committers 1
 - Rich Trott 
PR-URL: https://github.com/nodejs/node/pull/35108
Reviewed-By: Tobias Nießen 
Reviewed-By: Derek Lewis 
Reviewed-By: Franziska Hinkelmann 
------------------------------ Generated metadata ------------------------------
PR-URL: https://github.com/nodejs/node/pull/35108
Reviewed-By: Tobias Nießen 
Reviewed-By: Derek Lewis 
Reviewed-By: Franziska Hinkelmann 
--------------------------------------------------------------------------------
   ✔  Last GitHub Actions successful
   ✖  No Jenkins CI runs detected
   ℹ  This PR was created on Tue, 08 Sep 2020 22:09:45 GMT
   ✔  Approvals: 3
   ✔  - Tobias Nießen (@tniessen) (TSC): https://github.com/nodejs/node/pull/35108#pullrequestreview-484529575
   ✔  - Derek Lewis (@DerekNonGeneric): https://github.com/nodejs/node/pull/35108#pullrequestreview-484604383
   ✔  - Franziska Hinkelmann (@fhinkel) (TSC): https://github.com/nodejs/node/pull/35108#pullrequestreview-484744498
   ℹ  This PR is being fast-tracked
--------------------------------------------------------------------------------
   ✔  Aborted `git node land` session in /home/runner/work/node/node/.ncu

@github-actions github-actions bot added the commit-queue-failed An error occurred while landing this pull request using GitHub Actions. label Sep 10, 2020
@richardlau
Copy link
Member

Landed in 661b770.

richardlau pushed a commit that referenced this pull request Sep 10, 2020
A previous change altered the rendering of visited links. This restores
it.

PR-URL: #35108
Reviewed-By: Tobias Nießen <tniessen@tnie.de>
Reviewed-By: Derek Lewis <DerekNonGeneric@inf.is>
Reviewed-By: Franziska Hinkelmann <franziska.hinkelmann@gmail.com>
@richardlau richardlau closed this Sep 10, 2020
@richardlau richardlau mentioned this pull request Sep 10, 2020
4 tasks
richardlau pushed a commit that referenced this pull request Sep 10, 2020
A previous change altered the rendering of visited links. This restores
it.

PR-URL: #35108
Reviewed-By: Tobias Nießen <tniessen@tnie.de>
Reviewed-By: Derek Lewis <DerekNonGeneric@inf.is>
Reviewed-By: Franziska Hinkelmann <franziska.hinkelmann@gmail.com>
addaleax pushed a commit that referenced this pull request Sep 22, 2020
A previous change altered the rendering of visited links. This restores
it.

PR-URL: #35108
Reviewed-By: Tobias Nießen <tniessen@tnie.de>
Reviewed-By: Derek Lewis <DerekNonGeneric@inf.is>
Reviewed-By: Franziska Hinkelmann <franziska.hinkelmann@gmail.com>
addaleax pushed a commit that referenced this pull request Sep 22, 2020
A previous change altered the rendering of visited links. This restores
it.

PR-URL: #35108
Reviewed-By: Tobias Nießen <tniessen@tnie.de>
Reviewed-By: Derek Lewis <DerekNonGeneric@inf.is>
Reviewed-By: Franziska Hinkelmann <franziska.hinkelmann@gmail.com>
@codebytere codebytere mentioned this pull request Sep 28, 2020
joesepi pushed a commit to joesepi/node that referenced this pull request Jan 8, 2021
A previous change altered the rendering of visited links. This restores
it.

PR-URL: nodejs#35108
Reviewed-By: Tobias Nießen <tniessen@tnie.de>
Reviewed-By: Derek Lewis <DerekNonGeneric@inf.is>
Reviewed-By: Franziska Hinkelmann <franziska.hinkelmann@gmail.com>
@targos targos removed the commit-queue-failed An error occurred while landing this pull request using GitHub Actions. label Sep 5, 2021
@Trott Trott deleted the green branch April 14, 2022 11:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
doc Issues and PRs related to the documentations. fast-track PRs that do not need to wait for 48 hours to land.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

10 participants