mirror of
https://github.com/benbusby/whoogle-search.git
synced 2026-04-25 12:15:50 +03:00
[PR #956] [MERGED] Add calculator widget #1029
Labels
No labels
Fixed (Pending PR Merge)
Stale
bug
enhancement
enhancement
good first issue
help wanted
keep-open
needs more info
pull-request
question
theme
unfortunate
No milestone
No project
No assignees
1 participant
Notifications
Due date
No due date set.
Dependencies
No dependencies set.
Reference
starred/whoogle-search#1029
Loading…
Add table
Add a link
Reference in a new issue
No description provided.
Delete branch "%!s()"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
📋 Pull Request Information
Original PR: https://github.com/benbusby/whoogle-search/pull/956
Author: @ellieeet123
Created: 2/13/2023
Status: ✅ Merged
Merged: 2/21/2023
Merged by: @benbusby
Base:
main← Head:main📝 Commits (3)
ea9e73dAdd calculator widget6de6f05Specify font color for light themesa06f2bfFix/keyboard shortcut in calculator📊 Changes
6 files changed (+348 additions, -53 deletions)
View changed files
📝
app/routes.py(+13 -5)📝
app/static/js/keyboard.js(+4 -0)➕
app/static/widgets/calculator.html(+260 -0)📝
app/utils/results.py(+0 -38)📝
app/utils/search.py(+6 -10)➕
app/utils/widgets.py(+65 -0)📄 Description
This PR adds a simple calculator widget, somewhat similar to the one presented when searching
calculatoron Google. It is definitely not as fancy as Google's, and probably has a couple bugs that I wasn't able to find, but it works for the most part.Also, it adds somewhat of a template for making the addition of new widgets easier via the
app/utils/widgets.pyfile. My eventual plan is to use this to create more widgets that appear in Google, such as a color picker, timer, etc.Also as you probably have noticed this is my first PR here and I'm not too familiar with the code base. So if you see anything that doesn't look up to standards or out of place feel free to let me know.
Thanks!
🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.