Skip to content
This repository has been archived by the owner on Dec 11, 2019. It is now read-only.

Feature request: remove pages from history via the URL bar #1458

Closed
snird opened this issue Apr 24, 2016 · 4 comments
Closed

Feature request: remove pages from history via the URL bar #1458

snird opened this issue Apr 24, 2016 · 4 comments

Comments

@snird
Copy link

snird commented Apr 24, 2016

What:
Removing a page from history by hovering (or marking with arrow keys) the page in the URL bar suggestions, and pressing the DEL key.

Why:
Sometimes, while writing in the URL bar, there is a page we want to remove from history.
This is common behavior in other browsers (Firefox, Chrome..)

I know the changes should be on the URL bar suggestions component: https://github.com/brave/browser-laptop/blob/master/js/components/urlBarSuggestions.js
But I couldn't find the code responsible for sites history, and try to make them talk. Can someone give me a pointer?

@luixxiul
Copy link
Contributor

@bbondy please give him a hint :-)

@bbondy
Copy link
Member

bbondy commented May 14, 2016

History items there are stored in the application state sites list. They are the items with no tags.
I think you'd need to implement js/state/siteUtil's removeSite so when it is called with no tag, and it has no tags left, then it removes the site.

appActions.removeSite is what needs to be called.

@luixxiul
Copy link
Contributor

luixxiul commented Aug 2, 2016

I think this has been implemented. Correct me if I am wrong /cc: @bbondy

@bbondy
Copy link
Member

bbondy commented Aug 2, 2016

yep we have this

@bbondy bbondy closed this as completed Aug 2, 2016
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

3 participants