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

Highlight “arrows” in the Electron Energy Level Diagram #64

Closed
DianaTavares opened this issue Sep 23, 2024 · 1 comment
Closed

Highlight “arrows” in the Electron Energy Level Diagram #64

DianaTavares opened this issue Sep 23, 2024 · 1 comment
Assignees

Comments

@DianaTavares
Copy link
Contributor

2 teachers (from 9) during the feedback process commented that is challenging to see the arrows in the electron energy diagram (specially the gray-ones that correspond to UV photons):
image

Those arrows change their colors depending on the photon, so it should stay gray, red, etc. To solve this problem, the arrow head could be bigger, and also I think that the stroke can be a little bigger.

But, I see that the in the old mockups, that panel is black, that in general will make all the colors more contrasting:
image

The problem is that is not a solution for the projector mode, where the panels should be white. For me, it is difficult to play with those mockups, and draw lines with different curves as the sim proposes (because each arrow also represents the wavelengths with different numbers of curses). Then I am going to wait until @pixelzoom implement the panel in the sim, and later play with the colors of the panel,the strokes and the arrows heads.

@pixelzoom
Copy link
Contributor

pixelzoom commented Dec 17, 2024

This issue overlaps with #33, so I'm going to close this issue, and we can continue fine tuning the Electron Energy Level diagrams in #33. Specifically, #33 (comment) describes problems with the arrow head, and recommends getting rid of it altogether.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants