Skip to content

Commit 406f66b

Browse files
authored
Refactor popup removal logic and clean up SVG icons (#455)
Renamed parameter in remove_saved_page for clarity and fixed a redundant comparison in Popup.tsx. Removed duplicate or unnecessary SVG markup from code-web-chat-logo.svg, kimi.svg, and perplexity.svg to clean up icon assets.
1 parent cf83dab commit 406f66b

4 files changed

Lines changed: 3 additions & 40 deletions

File tree

‎packages/browser/src/views/popup/Popup.tsx‎

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -88,10 +88,10 @@ export const Popup: React.FC = () => {
8888
}
8989
}
9090

91-
const remove_saved_page = async (url: string) => {
92-
const success = await websites_store_hook.delete_website(url)
91+
const remove_saved_page = async (url_to_remove: string) => {
92+
const success = await websites_store_hook.delete_website(url_to_remove)
9393
if (success) {
94-
if (url == url) {
94+
if (url_to_remove == url) {
9595
set_is_saved(false)
9696
}
9797
await load_saved_websites()

‎packages/ui/src/assets/icons/code-web-chat-logo.svg‎

Lines changed: 0 additions & 17 deletions
Loading

‎packages/ui/src/assets/icons/kimi.svg‎

Lines changed: 0 additions & 12 deletions
Loading

‎packages/ui/src/assets/icons/perplexity.svg‎

Lines changed: 0 additions & 8 deletions
Loading

0 commit comments

Comments
 (0)