| FazBrowse GitHub Viewer | Trending | | Home |
| Tools: [Download Repo ZIP] [Original HTTPS Page] |
|
Oh I see that https://fastapi.tiangolo.com/ uses the same tooltips. Should I create a PR there too, or is this intentional and not a mistake? |
Sorry, something went wrong.
|
Looks similar if not the same? fastapi/fastapi#3246 |
Sorry, something went wrong.
|
Maybe swapping the icons would be better too, sure. While using the light theme the bulb would be "turned on" and with the dark theme it would be "off". Makes sense to me. However fastapi/fastapi#3246 does not fix the tooltip texts. |
Sorry, something went wrong.
|
Anyway, swapping the text alone seems like an improvement for me. Swapping the icons can be something for another PR. |
Sorry, something went wrong.
Codecov Report
@@ Coverage Diff @@
## main #1 +/- ##
=======================================
Coverage ? 97.27%
=======================================
Files ? 175
Lines ? 5842
Branches ? 0
=======================================
Hits ? 5683
Misses ? 159
Partials ? 0 Continue to review full report at Codecov.
|
Sorry, something went wrong.
|
Great, thank you @Lehoczky! 🚀 And thanks for the comments @ArcLightSlavik! |
Sorry, something went wrong.
| Back | FazBrowse Home | New Git URL |
Just a simple fix for the lightbulb's tooltip on the documentation site:
Before:


After:

