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

unable to open new window from dock icon #13860

Closed
LaurenWags opened this issue Apr 18, 2018 · 4 comments
Closed

unable to open new window from dock icon #13860

LaurenWags opened this issue Apr 18, 2018 · 4 comments

Comments

@LaurenWags
Copy link
Member

LaurenWags commented Apr 18, 2018

Test Plan

#13866
Also follow STR below if applicable for your platform.

Description

When Brave is running, but there aren't any open windows, you used to be able to click on the dock icon to open a new window. This no longer works.

Reported via community:
https://community.brave.com/t/browser-version-0-22-13-browser-window-not-opening-after-initial-open/19232
https://community.brave.com/t/updating-macos-from-0-22-13-to-0-22-21/19915/7

Steps to Reproduce

  1. Launch Brave.
  2. Close any open windows (Brave is still running, note black dot below the icon in the dock).
  3. Click on the Brave icon in the dock.

Actual result:
Nothing happens. It looks like Brave isn't working. Workaround: Navigate to File > New Window or right click on the icon in the dock and select New Window. New window displays.

User from community provided this video: https://www.youtube.com/watch?v=mb2ZmXpXw58&feature=youtu.be

Expected result:
If Brave is running but you don't have any open windows, clicking on the icon in the dock should open a new window.

Reproduces how often:
Easily

Brave Version

about:brave info:
0.22.22

Reproducible on current live release:
Yes, was last working in 0.21.24 and stopped working in 0.22.13

Additional Information

Seems like maybe it could be related to #13689 but the fix for this one does not resolve the reported issue (checked with 0.22.109)

@petemill petemill self-assigned this Apr 19, 2018
@petemill petemill added this to the 0.22.x w/ Chromium 66 milestone Apr 19, 2018
petemill added a commit that referenced this issue Apr 19, 2018
…cused

Fix #13859
Fix #13860
Fix #13865

Makes sure tabs are created in a shown window, or creates a new window, that the window is focused, and if the shown window is minimized, then that window is restored.
Fixes for the following scenarios...:
    - There are no windows open
    - There are only minimized windows open
...For the following actions:
    - Brave is default browser and link clicked in another application
    - macOS dock icon is clicked
    - 'Brave' menu actions such as 'New Tab' or 'About Brave'

Test plan:
Browser must be set as default. If testing from source, the best way is to create a build `CHANNEL=dev npm run build-package`, run that, and set it as default.
    - Click URL link in external application
      - Ensure that tab opens in an active window, and the window is focused with the following scenarios:
        - 1 window open
        - 2+ windows open
        - 0 windows open
        - 1 minimized window open
        - 1 minimized window open and 1 non-minimized window open
        - Application exited
      - macOS: click dock icon, ensure window is focused or created:
        - 1 window open
        - 2+ windows open
        - 0 windows open
        - 1 minimized window open
        - 1 minimized window open and 1 non-minimized

Even though the reported issues were macOS only, this test plan should be carried out on other platforms to ensure no regressions.
@LaurenWags
Copy link
Member Author

LaurenWags commented Apr 20, 2018

Verified on macOS 10.12.6 with
0.22.665 bc13ab6
Muon 5.2.5
libchromiumcontent 66.0.3359.117

Verified on Windows x64

  • 0.22.667 0433b26
  • libchromiumcontent 66.0.3359.117
  • muon: 5.2.6

Verified on Ubuntu 17.10
0.22.667 0433b26
Muon 5.2.6
libchromiumcontent 66.0.3359.117

@LaurenWags
Copy link
Member Author

Removed macOS label - linked test plan was recommended to be checked on all platforms.

@bsclifton
Copy link
Member

+1 from my dad on this one. Thanks for the fix, @petemill 😄👍

@j-peeters
Copy link

Thanks for fixing this!

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

6 participants