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

Bookmarked about pages appear in the history #3366

Closed
luixxiul opened this issue Aug 24, 2016 · 2 comments · Fixed by #3459
Closed

Bookmarked about pages appear in the history #3366

luixxiul opened this issue Aug 24, 2016 · 2 comments · Fixed by #3459
Assignees
@bsclifton
Labels
feature/history QA/checked-Linux QA/checked-macOS QA/checked-Win32 QA/checked-Win64
Milestone
0.11.6dev

Comments

@luixxiul
Copy link
Contributor

luixxiul commented Aug 24, 2016 •
edited
Loading

When I bookmark about pages, they appear on about:history.

  • Platform (Win7, 8, 10? macOS? Linux distro?): Windows 7 64 bit
  • Brave Version: 0.11.6 beta3
  • Steps to reproduce:
    1. Open about:history from url bar
    2. bookmark it with CTRL+D and clicking "Save" on the dialog
    3. Open a new tab
    4. Go to github.com
    5. Select the about:history tab

The same thing happens when I open and bookmark about:passwords.

  • Screenshot if needed:
    clipboard01

I am not sure if this is intended and it works as it should or not.

cc: @bsclifton
#3362
@bbondy
Copy link
Member

bbondy commented Aug 25, 2016

I think Chrome filters out all about pages, please verify and if so do the same here.

@bsclifton bsclifton self-assigned this Aug 26, 2016
@bsclifton bsclifton mentioned this issue Aug 26, 2016
Merged
3 tasks
@bsclifton
Copy link
Member

Fixed w/ #3459

@luixxiul luixxiul added this to the 0.11.6dev milestone Aug 26, 2016
This was referenced Aug 27, 2016
Closed
Closed
Closed
@alexwykoff alexwykoff mentioned this issue Aug 31, 2016
Closed
@alexwykoff alexwykoff mentioned this issue Aug 31, 2016
Closed
@aekeus aekeus mentioned this issue Aug 31, 2016
Closed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@bsclifton bsclifton

Labels
feature/history QA/checked-Linux QA/checked-macOS QA/checked-Win32 QA/checked-Win64
Projects
None yet
Milestone
0.11.6dev
Development

Successfully merging a pull request may close this issue.

History bugfixes
4 participants
@bbondy @alexwykoff @luixxiul @bsclifton