search

TinyTapeout / siliwiz

Silicon Layout Wizard
https://app.siliwiz.com
Other
141 stars 10 forks source link

low contrast between ntap & nwell colors #43

Open mbalestrini opened 1 year ago

mbalestrini commented 1 year ago

The n-tap drawigs inside n-wells are not clearly visible. It would be good to use another color that makes the n-taps popup a little bit more.

urish commented 1 year ago

Thanks! Any suggestions for a new color? The colors are defined in src/model/layerTypes.ts

mbalestrini commented 1 year ago

I changed the colors of nwell and substrate and made a pull request: #52 I've included and image so you can check if you like the changes

urish commented 1 year ago

Thanks @mbalestrini! I've had a discussion about this with @mattvenn, and changing the color palette means he'll have to redo the screen shots in the lesson plans.

There's also another change which might affect the color palette at #25 (you can preview it here: https://layer-opacity.siliwiz.pages.dev/).

Therefore, we'd like to have a color palette that includes both your changes and opacity. I don't know if you'd like to work on updating the color palette to include the opacity as well - in case you do, that would be helpful.

mbalestrini commented 1 year ago

I'll take a look at the opacity idea. I'm not 100% sure about it. I think it might make everything more confuse for someone learning? I'll try to do some tests in inkscape to see how it might look