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

Context menu now works on sortableTable with no selection #5206

Merged
merged 2 commits into from
Oct 28, 2016
Merged

Context menu now works on sortableTable with no selection #5206

merged 2 commits into from
Oct 28, 2016

Conversation

bsclifton
Copy link
Member

@bsclifton bsclifton commented Oct 27, 2016

  • Submitted a ticket for my issue if one did not already exist.
  • Used Github auto-closing keywords in the commit message.
  • Added/updated tests for this change (for new code or code which already has tests).
  • Ran git rebase -i to squash commits (if needed).

Context menu now works on sortableTable with no selection
Also fixes: focus for new folder / edit folder / new bookmark now set properly

Fixes #5202

Auditors: @darkdh

Test plan:

  1. Launch Brave and go to about:bookmarks
  2. Without selecting an item, right click on any entry to bring up context menu
  3. Confirm you see the context menu for a single item
  4. Visit about:history
  5. without selecting an item, right click on any entry to bring up context menu
  6. Confirm you see the context menu for a single item

@bsclifton bsclifton added this to the 0.12.7dev milestone Oct 27, 2016
Also fixes: focus for new folder / edit folder / new bookmark now set properly

Fixes #5202

Auditors: @darkdh
@bsclifton bsclifton changed the title Fixes for 0.12.7 testing Context menu now works on sortableTable with no selection Oct 27, 2016
@@ -16,9 +16,13 @@ class AddEditBookmark extends ImmutableComponent {
onClose () {
windowActions.setBookmarkDetail()
}
componentDidMount () {
Copy link
Member Author

@bsclifton bsclifton Oct 27, 2016

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this and the change in addEditBookmarkHanger.js are changes I missed when originally doing that control, which was submitted with #5118

Focus is now properly set to the title input box when the modal launches (no issue had been filed yet)

Auditors: @bsclifton

Test Plan:
1. go to "about:history"
2. do not select anything
3. right click on an arbitrary entry
4. context menu should show
@darkdh
Copy link
Member

darkdh commented Oct 28, 2016

++ to 0e6ba28

please review d41a2d8

@bsclifton
Copy link
Member Author

++ thanks for the assist! 😄

@luixxiul
Copy link
Contributor

luixxiul commented Oct 28, 2016

Did this fix #4811 too? It seems so.

@bsclifton
Copy link
Member Author

@luixxiul it sure does! thanks 😄 I went ahead and closed #4811

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
5 participants