Skip to content
This repository has been archived by the owner. It is now read-only.

Change windows in appState from List to Map #11151

Closed
NejcZdovc opened this issue Sep 26, 2017 · 0 comments
Closed

Change windows in appState from List to Map #11151

NejcZdovc opened this issue Sep 26, 2017 · 0 comments

Comments

@NejcZdovc
Copy link
Member

@NejcZdovc NejcZdovc commented Sep 26, 2017

Description

We want to have windowId as a key in windows. This way you can easily access windows without needing to use find function. Be carful when doing this, because windowId changes when you re-launch browser, so id's needs to be set on every launch.

Steps to Reproduce

Actual result:

Expected result:

Reproduces how often: [What percentage of the time does it reproduce?]

Brave Version

about:brave info:

Reproducible on current live release:

Additional Information

@NejcZdovc NejcZdovc added this to the 0.21.x (Nightly Channel) milestone Sep 26, 2017
@ghost ghost added the priority/P3 label Sep 26, 2017
@ghost ghost assigned NejcZdovc Sep 26, 2017
@bbondy bbondy modified the milestones: 0.21.x (Developer Channel), 0.20.x (Beta Channel), Backlog Oct 25, 2017
@bbondy bbondy modified the milestones: Triage Backlog, Prioritized Backlog Nov 2, 2017
@bsclifton bsclifton mentioned this issue Jan 9, 2018
0 of 9 tasks complete
@NejcZdovc NejcZdovc removed their assignment Jan 12, 2018
@bsclifton bsclifton added the stale label Sep 1, 2018
@bsclifton bsclifton closed this Sep 1, 2018
@bsclifton bsclifton removed this from the Backlog (Prioritized) milestone Sep 1, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
4 participants
@bbondy @bsclifton @NejcZdovc and others
You can’t perform that action at this time.