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

Split bookmark manager component and move it into app folder #10075

Closed
NejcZdovc opened this issue Jul 20, 2017 · 0 comments
Closed

Split bookmark manager component and move it into app folder #10075

NejcZdovc opened this issue Jul 20, 2017 · 0 comments
Assignees
@NejcZdovc
Labels
feature/bookmarks QA/checked-Linux QA/checked-macOS QA/checked-Win64 QA/test-plan-specified refactoring release-notes/exclude
Milestone
0.20.x (Release C...

Comments

@NejcZdovc
Copy link
Contributor

NejcZdovc commented Jul 20, 2017 •
edited by luixxiul
Loading

Test plan

#10074 (comment)

Describe the issue you encountered:
Split js/about/bookmarks.js into separate components and move them into app folder
@NejcZdovc NejcZdovc added this to the 0.20.x (Developer Channel) milestone Jul 20, 2017
@NejcZdovc NejcZdovc self-assigned this Jul 20, 2017
NejcZdovc added a commit to NejcZdovc/browser-laptop that referenced this issue Jul 20, 2017
@NejcZdovc
Moves bookarks manager from js to app
bc06f35 
Resolves brave#10075

Auditors: @bsclifton

Test Plan:
- check if bookmarks manager is displayed correctly
@NejcZdovc NejcZdovc mentioned this issue Jul 20, 2017
Merged
8 tasks
This was referenced Dec 20, 2017
Closed
Closed
This was referenced Dec 27, 2017
Closed
Closed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@NejcZdovc NejcZdovc

Labels
feature/bookmarks QA/checked-Linux QA/checked-macOS QA/checked-Win64 QA/test-plan-specified refactoring release-notes/exclude
Projects
None yet
Milestone
0.20.x (Release Channel)
Development

No branches or pull requests
4 participants
@luixxiul @NejcZdovc @srirambv @LaurenWags