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

[WiP] convert toolbar icons to svg #4890

Closed
wants to merge 59 commits into from
Closed

[WiP] convert toolbar icons to svg #4890

wants to merge 59 commits into from
Commits on Aug 29, 2016
  1. Start suggestions after first character not second fixes #3235

    Radoslav Vitanov
    Radoslav Vitanov committed Aug 18, 2016
Commits on Sep 4, 2016
  1. Merge branch 'master' into hotfix/url-bar-suggestions

    Radoslav Vitanov
    Radoslav Vitanov committed Sep 4, 2016
Commits on Sep 6, 2016
  1. Merge branch 'master' into hotfix/url-bar-suggestions

    Radoslav Vitanov
    Radoslav Vitanov committed Sep 6, 2016
Commits on Oct 13, 2016
  1. Merge branch 'master' into hotfix/url-bar-suggestions

    Radoslav Vitanov
    Radoslav Vitanov committed Oct 13, 2016
Commits on Oct 15, 2016
  1. UrlbarSuggestions does not work if you are offline

    Radoslav Vitanov
    Radoslav Vitanov committed Oct 15, 2016
    Fixes #3730
    
    Auditors:
    @bbondy
    
    Test Plan:
    1. Make sure you turned on every type of suggestions you want to get via typing from the settings menu and set default Search Engine to duckduckgo
    2. Disconnect from internet
    3. Start typing something in the url bar
Commits on Oct 17, 2016
  1. Disable save button when no detail present

    darkdh committed Oct 17, 2016
    fix #4855
    
    Auditors: @bbondy
    
    Test Plan:
    1. Open "about:autofill"
    2. Click "Add Address"
    3. The save button should be disabled
    4. Enter the detail of form, the save button should be enabled
    5. Remove all detail of form, the save button should be disabled
    
    1. Open "about:autofill"
    2. Click "Add Credit Card"
    3. The save button should be disabled
    4. Enter the detail of form, the save button should be enabled (Name, Card Number)
    5. Remove all detail of form, the save button should be disabled (Name, Card Number)
    
    1. Open "about:autofill"
    2. Add address entry
    3. Edit the address entry
    4. Remove all detail of form, the save button should be disabled
    
    1. Open "about:autofill"
    2. Add credit card entry
    3. Edit the credit card entry
    4. Remove all detail of form, the save button should be disabled (Name, Card Number)
Commits on Oct 18, 2016
  1. "New bookmark folder" and "New bookmark" buttons added to about:bookm…

    bsclifton committed Oct 18, 2016
    …arks
    
    - sortableTable now allows you to provide a control instead of a string for the header
    - about:bookmarks now properly sets tableID (not used yet)
    - Adding a new bookmark now requires either a title or location (URL). Required because clicking new bookmark creates an empty object (versus existing buttons which always supply the current page).
    - Fixed style issue with about:bookmarks (moved border-right on folderView to organizeView
    
    Fixes #4684
    Fixes #4866
    
    Auditors: @jkup, @bbondy
    
    Test Plan:
    1. Launch Brave and visit about:bookmarks
    2. Pick any folder on the left hand side and click it to select it
    3. Click the +folder icon, above the folder view
    4. Confirm that the new bookmark folder UI comes up and that it sets the parent folder properly
    5. Click add and ensure folder gets added
    6. Pick any folder and click it to select
    7. In the "title" column, hit the star+ icon to add a new bookmark
    8. Confirm you can't add an empty bookmark. Type and see the button enable itself. Delete chars and it disables itself again.
    9. Confirm the parent folder is correct and type in a title/url. When done, hit save.
    10. Confirm bookmark was added.
  2. Update localization

    Suguru Hirahara
    Suguru Hirahara committed Oct 18, 2016
    Auditors: @aekeus
  3. convert toolbar icons to svg

    jkup committed Oct 18, 2016
  4. fix Brave crash on confirm(1) bug

    willy-b committed Oct 18, 2016
    - make sure that when `message` is a  number (or other non-string type) it is coerced to a string
    - same change as what @bridiver did for alerts in #4798, but for confirm dialog
    
    fixes #4883
  5. Also fix crashes when alert/confirm titles not strings

    willy-b committed Oct 18, 2016
    Also fix crashes for URLs like `data:text/html,<script>alert\(1, 1\)</script>`
    and `data:text/html,<script>confirm\(1, 1\)</script>`
  6. Merge pull request #4875 from darkdh/4855

    bbondy committed Oct 18, 2016
    Disable save button when no detail present
  7. Merge pull request #4888 from bsclifton/bm-manager-add-folder

    bbondy committed Oct 18, 2016
    "New bookmark folder" and "New bookmark" buttons added to about:bookmarks
  8. Merge pull request #4889 from luixxiul/transifex-update

    bbondy committed Oct 18, 2016
    Update localization
  9. Update macOS dmg background to use new icon

    bbondy committed Oct 18, 2016
    Audtors: @bsclifton
    
    Fix #4817
  10. Allow about pages to reload and clone

    darkdh committed Oct 18, 2016
    fix #2826
    
    Auditors: @bridiver, @bbondy
    
    Test Plan:
    1. Generate any error pages(connection error, cert error, ...)
    2. Clone/Reload should work properly (reload the original URL)
    
    1. Go to any about pages
    2. Clone/Reload should work properly (reload the original URL)
  11. Merge pull request #4881 from darkdh/2826

    bbondy committed Oct 18, 2016
    Allow about pages to reload and clone
  12. Merge pull request #4821 from Sh1d0w/hotfix/issue-3730

    bbondy committed Oct 18, 2016
    UrlbarSuggestions does not work if you are offline
  13. Merge pull request #3238 from Sh1d0w/hotfix/url-bar-suggestions

    bbondy committed Oct 18, 2016
    Start suggestions after first character not second fixes #3235
  14. updated release notes for 0.12.5 RC6

    alexwykoff committed Oct 18, 2016
  15. Merge pull request #4901 from brave/0_12_5_rc6_update_release_notes

    bbondy committed Oct 18, 2016
    updated release notes for 0.12.5 RC6
  16. Caption buttons are no longer tab focusable

    bsclifton committed Oct 18, 2016
    Fixes #4846
    
    Auditors: @bbondy
    
    Test plan:
    1. Launch Brave on Windows and set focus to URL bar
    2. Press tab to move focus and hit enter
    3. Window should not minimize, maximize, or close (focus should have went
    to the page)
  17. Do not use default setting when value is false

    darkdh committed Oct 18, 2016
    fix #4903
    
    Auditors: @bbondy
    
    Test Plan:
    1. Turn on/off "Always show the URL bar" in about:preferences#general
    2. The title mode should work as expected
  18. Merge pull request #4907 from darkdh/4903

    bbondy committed Oct 18, 2016
    Do not use default setting when value is false
  19. fix typo

    bsclifton committed Oct 18, 2016
  20. Merge pull request #4902 from bsclifton/win-tab-focus

    bbondy committed Oct 18, 2016
    Caption buttons are no longer tab focusable
  21. Remove some redundant lines

    willy-b committed Oct 18, 2016
  22. History page now properly opens new tab w/ link that was clicked

    bsclifton committed Oct 18, 2016
    Fixes #4909
    
    Auditors: @darkdh
    
    Test Plan:
    1. Launch Brave and visit a few sites
    2. Visit about:history
    3. Double click one of the entries and confirm it opens the location in a new tab
  23. Merge pull request #4919 from bsclifton/fix-history-doubleclick

    darkdh committed Oct 18, 2016
    History page now properly opens new tab w/ link that was clicked
  24. Move bookmark hanger arrow now that reload is on about pages

    jkup committed Oct 18, 2016
  25. URL input field on the bookmark pulldown disappears

    jkup committed Oct 18, 2016
Commits on Oct 19, 2016
  1. Merge pull request #4891 from willy-b/issue-4883-2

    bbondy committed Oct 19, 2016
    Fix Brave crashing on confirm bug
  2. Merge pull request #4927 from brave/bookmark-pulldown-disappearing

    bbondy committed Oct 19, 2016
    URL input field on the bookmark pulldown disappears
  3. Merge pull request #4924 from brave/bookmark-hanger-arrow

    bbondy committed Oct 19, 2016
    Move bookmark hanger arrow now that reload is on about pages
  4. Import favicon fix:

    darkdh committed Oct 19, 2016
    1. Iterate all urls using the favicon
    2. Adjust made-up-favicon condition and override old made-up-favicon url
    
    fix #4882
    
    Auditors: @bridiver, @bbondy
    
    Test Plan:
    1. Import bookmarks from firefox on brave 0.12.3
    2. Import bookmarks from firefox on brave contains this commit
    3. The bookmarks under "Mozilla Firefox" should have favicons
    
    1. Import bookmarks from any browsers
    2. The imported bookmarks should have favicons
    
    1. Import bookmarks from html bookmarks file
    2. The imported bookmarks should have favicons
  5. Upgrade to brave/electron 1.4.15

    bbondy committed Oct 19, 2016
    Fix #4932
    
    Auditors: @bridiver
  6. Add whether brave is default browser check

    darkdh committed Oct 14, 2016
    fix #2105
    requires brave/muon#73
    
    Auditors: @bridiver, @bbondy
  7. 0.12.6

    bbondy committed Oct 19, 2016
    Auditors: @alexwykoff
  8. Merge pull request #4872 from brave/default-browser

    bbondy committed Oct 19, 2016
    Add whether brave is default browser check
  9. Delete "defaultBrowserCheckComplete" only when check on startup is on

    darkdh committed Oct 19, 2016
    fix #4934
    
    Auditors: @bbondy
    
    Test Plan:
    1. Turn off always check on startup
    2. Close app
    3. Open app
    4. Go to preferences
    5. Toggle the switch for always check on startup
  10. Merge pull request #4942 from darkdh/4934

    bbondy committed Oct 19, 2016
    Delete "defaultBrowserCheckComplete" only when check on startup is on
  11. Split apart "always show bookmarks toolbar" and the bookmarks toolbar…

    bsclifton committed Oct 19, 2016
    … display mode
    
    Fixes #4936
    
    Auditors: @bbondy
    
    Test Plan:
    1. Launch Brave and open Preferences
    2. Verify you can toggle "always show bookmarks toolbar" and not have the drop down above be affected
  12. Merge pull request #4944 from bsclifton/bookmark-settings-breakup

    bbondy committed Oct 19, 2016
    Split apart "always show bookmarks toolbar" and the bookmarks toolbar display mode
  13. Improve UX of default browser check and Manage Adblock Setting button

    darkdh committed Oct 19, 2016
    fix #4935
    
    Auditors: @bbondy
    
    Test Plan: N/A
  14. Merge pull request #4951 from darkdh/4935

    bbondy committed Oct 19, 2016
    Improve UX of default browser check and Manage Adblock Setting button
  15. New icons

    bbondy committed Oct 19, 2016
    Auditors: @bsclifton
    
    Fix #4120
  16. Upgrade to brave/electrton 1.4.16

    bbondy committed Oct 19, 2016
    Fix #4954
  17. Merge pull request #4930 from darkdh/4882

    bbondy committed Oct 19, 2016
    Import favicon fix:
  18. update to ledger-client@0.8.89 / ledger-publisher@0.8.89

    mrose17 committed Oct 19, 2016
  19. Revert "Prevent clickjacking on about: pages"

    bbondy committed Oct 18, 2016
    This reverts commit 878f7c5.
  20. Limit web_accessible_resources to about:flash and about:blank

    diracdeltas committed Oct 19, 2016
    Fix #4913
    Fix #4885
    
    Auditors: @bbondy
    
    Test Plan:
    1. go to homestarrunner.com and verify that the flash placeholder appears
    2. go to http://web.mit.edu/zyan/Public/xframe.html and verify that the iframe is blank
    3. open page devtools, load about:preferences, and verify in the Network tab that the `Access-Control-Allow-Origin` response header is not present on about-preferences.html
  21. explicitly declare punycode dependency

    diracdeltas committed Oct 19, 2016
    As of nodejs/node#7941, the node core punycode
    module is deprecated.
    
    Auditors: @bbondy
    
    Test Plan: n/a
  22. Show punycode URL in homepage

    diracdeltas committed Oct 19, 2016
    Fix #4747
    
    Auditors: @bbondy
    
    Test Plan: covered by automated test
  23. Fixes #4018

    jonathansampson committed Oct 19, 2016
    Fixes issue where LastPass' popup window is closed during user attempt
    to shift focus from username field to password field.
    
    Auditor: @bridiver
  24. default browser check should run only on release build and

    darkdh committed Oct 19, 2016
    BRAVE_IS_DEFAULT_BROWSER for test purpose
    
    fix #4958
    
    Auditors: @bridiver, @bbondy
    
    Test Plan:
    1. Make sure there is no session data
    2. npm run start/test should not popup the check default browser modal
    and overrides brave as default browser.
    3. BRAVE_IS_DEFAULT_BROWSER=true npm run start/test should not popup the check default browser modal
    and overrides brave as default browser.
    4. BRAVE_IS_DEFAULT_BROWSER=false npm run start/test should popup the check default browser modal
    and overrides brave not be default browser.
  25. Merge pull request #4964 from jonathansampson/lastpass_focus

    bridiver committed Oct 19, 2016
    Fixes #4018
  26. Merge pull request #4959 from darkdh/4958

    bridiver committed Oct 19, 2016
    default browser check should run only on release build
Commits on Oct 20, 2016
  1. convert toolbar icons to svg

    jkup committed Oct 18, 2016
  2. Merge branch 'icons-to-svg' of github.com:brave/browser-laptop into i…

    jkup committed Oct 20, 2016
    …cons-to-svg
You can’t perform that action at this time.