Skip to content

x/website: Cursor color on the Tour of Go editor in dark mode is not readable #63313

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

Closed
Manish-Giri opened this issue Sep 30, 2023 · 2 comments
Closed
Assignees
Labels
FrozenDueToAge NeedsFix The path to resolution is known, but the work has not been done. pkgsite website

Comments

@Manish-Giri
Copy link

What is the URL of the page with the issue?

https://go.dev/tour/welcome/1

What is your user agent?

Mozilla/5.0 (Macintosh; Intel Mac OS X 10_15_7) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/117.0.0.0 Safari/537.36

Screenshot

Screenshot 2023-09-30 at 9 59 17 AM

Notice the barely-visible black cursor at the end of the fmt.Println() statement. It looks like the color was selected for the light mode and never adjusted for dark mode. Below is the same cursor in light mode.

Screenshot 2023-09-30 at 10 02 40 AM

What did you do?

  1. Visit tour of go.
  2. Turn on dark mode.

What did you expect to see?

A clearly visible cursor.

What did you see instead?

See screenshot above.

@gopherbot gopherbot added this to the Unreleased milestone Sep 30, 2023
AlexanderYastrebov added a commit to AlexanderYastrebov/golang-website that referenced this issue Sep 30, 2023
Add styles for dark theme cursor and selection.
Cursor color set equal to the foreground color.
Focused selection color set to the complement color
of the background color and inactive selection
is the lighter tint of it.

Fixes golang/go#63313
@AlexanderYastrebov
Copy link
Contributor

Created golang/website#253 to fix cursor and selection colors.

Screenshot from 2023-10-01 00-27-07
Screenshot from 2023-10-01 00-25-56
Screenshot from 2023-10-01 00-26-00

@gopherbot
Copy link
Contributor

Change https://go.dev/cl/531856 mentions this issue: _content/tour: add dark theme cursor and selection styles

@hyangah hyangah modified the milestones: Unreleased, website/backlog Oct 5, 2023
@hyangah hyangah added the NeedsFix The path to resolution is known, but the work has not been done. label Oct 5, 2023
willpoint pushed a commit to orijtech/website that referenced this issue Oct 17, 2023
Add styles for dark theme cursor and selection.
Cursor color set equal to the foreground color.
Focused selection color set to the complement color
of the background color and inactive selection
is the lighter tint of it.

Fixes golang/go#63313

Change-Id: If9702f45262eea330d42ec0fd9717b36da026540
GitHub-Last-Rev: 1545bba
GitHub-Pull-Request: golang#253
Reviewed-on: https://go-review.googlesource.com/c/website/+/531856
TryBot-Result: Gopher Robot <gobot@golang.org>
Auto-Submit: Suzy Mueller <suzmue@golang.org>
LUCI-TryBot-Result: Go LUCI <golang-scoped@luci-project-accounts.iam.gserviceaccount.com>
Run-TryBot: Hyang-Ah Hana Kim <hyangah@gmail.com>
Reviewed-by: Hyang-Ah Hana Kim <hyangah@gmail.com>
Reviewed-by: Suzy Mueller <suzmue@golang.org>
@golang golang locked and limited conversation to collaborators Oct 4, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
FrozenDueToAge NeedsFix The path to resolution is known, but the work has not been done. pkgsite website
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants