Commit Graph
36331 Commits
Author SHA1 Message Date
Joao Moreno 610629af3b remove unused code 2018-06-24 23:28:20 +02:00
Joao Moreno 2f224a3502 tree: hide twisties for files 2018-06-24 23:23:30 +02:00
Joao Moreno 0e38cbebb0 tree: quick demo 2018-06-24 23:20:32 +02:00
Joao Moreno 6e270a1ea5 tree: missing visibleCount update 2018-06-24 23:20:02 +02:00
Ronald Rey 5eea6bcc4d Add "Recommend" action to extension viewlet (#50419)
* Add "Recommend" action to extension viewlet

Related to #13456

* Address PR requested changes

* Change functionality and UX

- Button goes from "Add to workspace recommendations" (clickable) to
  "Adding to workspace recommendations..." (unclickable) to
  "Added to workspace recommendations" (unclickable).
- It is only visible if the current extension is installed and there's
  at least one folder in the root that doesn't have it recommended yet.
  In other words, if it's already recommended in every folder of the root,
  it is not visible.
- In a single-root setup, it's immediately added to the recommendations.
- In a multi-root setup, a quick-pick is displayed with only the folders
  where it is not yet recommended available as options.
- In a multi-root setup, the button will go back to
  "Add to workspace recommendations" (clickable) instead of the
  unclickable state if there are still folders remaining where the
  current extension is not yet recommended.
- An error or success notification is displayed after the work is done.
- Configuration files are created if don't exist, modified otherwise.

* Don't show the button if ext is recommended in ANY folder

In the previous commit, the button would be displayed as long as there
was at least one folder in the workspace where the extension wasn't
recommended.

Now, the button will be displayed only if the current extension is not
recommended in any of the folders, as suggested in
https://github.com/Microsoft/vscode/pull/50419#issuecomment-398216711

* Improvements based on PR suggestions

- Lowercasing the extension ID before comparing
- Directly styling & labeling the button on the `run` command after work is done instead of calling `update`.
- Fix & delete unnecessary styles

* Expose feature as command instead of button

After a discussion with @ramya-rao-a and other members of the team,
it was decided that it wasn't reasonable to have a dedicated button for
a feature that's rarely used in such a visible place.

So instead, the action will be exposed as a command that will only be
available when there's an extension open in the editor area. In contrast
with the previous implementation, this has the added benefit of
allowing the user to use this action to recommend uninstalled
extensions as well.

* Simplify conditions for AddToRecommendations action

* Move comments so that they dont get erased when recommendations are added programatically

* Remove from unwanted before adding to recommendations

* Array improvements

* Friendly error msg
2018-06-24 12:21:21 -07:00
Joao Moreno 0271c19d2c tree: iterators are optional 2018-06-24 20:44:23 +02:00
João Moreno ba69c69a83 Merge pull request #52765 from Microsoft/joao/win32-user-install
Win32 User Install
2018-06-24 20:42:59 +02:00
Joao Moreno 6002d769ea tree: make ITreeElement easier to use 2018-06-24 20:36:07 +02:00
Rob Lourens 3846925567 Settings editor - fix measuring items with wrong template 2018-06-24 11:28:26 -07:00
Rob Lourens 74ccd2dffa Fix #52758 - Don't add 'undefined' include/exclude keys for findFiles 2018-06-24 11:28:26 -07:00
Joao Moreno 4b3c04ec67 tree: expand/collapse on click 2018-06-24 20:24:35 +02:00
Joao Moreno 323bbc57ef tree: simply expose ITreeNode objects to list 2018-06-24 20:15:52 +02:00
Joao Moreno a1c2e80849 tree model: set parent chain 2018-06-24 20:09:43 +02:00
EbXpJ6bp 4ae06bd92f Improve Extension editor UI (#52126)
Improve Dependencies and Contributions UI in the Extension editor
2018-06-24 10:57:10 -07:00
Rob Lourens 21d654c742 Fix #52607 - expose refreshSearchResults as command so users can add keybindings 2018-06-23 20:43:43 -07:00
Rob Lourens dba25f9bee Fix #52584 - more descriptive label for "clear search results" button 2018-06-23 20:35:07 -07:00
Rob Lourens f2b1432b59 Settings editor - left-align reset button 2018-06-23 19:11:39 -07:00
Joao Moreno 1cdf1a7c9d more tree 2018-06-24 00:22:06 +02:00
Joao Moreno 83dd7007bc first tree showcase 2018-06-24 00:09:45 +02:00
Joao Moreno 44e89a0d64 tree: more wrapping code around list 2018-06-23 23:53:07 +02:00
Joao Moreno bddf469315 treeModel: setCollapsed 2018-06-23 23:26:55 +02:00
Leonardo Braga 605ba83864 Fixes stuck hover when manually triggered and "editor.hover" is false
- Adds support to toggle hover without reloading the editor
2018-06-23 15:52:36 -04:00
Andre Weinand 99a9b0a06d node-debug@1.25.3 2018-06-23 18:27:08 +02:00
Joao Moreno 0ddccb55ab 💄 2018-06-23 17:55:02 +02:00
Joao Moreno 053beaffe2 list > view 2018-06-23 17:54:47 +02:00
Joao Moreno ad7de10b2b tree: first sketch 2018-06-23 17:54:26 +02:00
Joao Moreno 51d3769600 tree: cleanup 2018-06-23 17:40:37 +02:00
Joao Moreno 0b6cfd536d tree model 💄 2018-06-23 16:49:32 +02:00
Joao Moreno 1ab82704c0 tree model: better error handling 2018-06-23 16:46:01 +02:00
Joao Moreno 34f080ebaf more tree model tests 2018-06-23 16:44:23 +02:00
Joao Moreno 842fdc7d52 unify tree models 2018-06-23 16:27:39 +02:00
Rob Lourens e7a2321e22 Settings editor - checkbox/textbox styling 2018-06-22 18:46:59 -07:00
Rob Lourens a5bfda035f Settings editor - 💄 2018-06-22 17:57:17 -07:00
Rob Lourens eaec01e005 Settings editor - force double-click open mode on TOC 2018-06-22 16:42:01 -07:00
Rob Lourens 8dd6ff6d92 Settings editor - sync scroll from settings tree to TOC tree 2018-06-22 16:29:52 -07:00
Joao Moreno a8e7ed08c6 first successful CollapsibleTreeModel tests, woo 2018-06-23 00:15:21 +02:00
Joao Moreno dd5e23bf15 CollapsibleTreeModel 2018-06-22 23:58:40 +02:00
Ramya Achutha Rao c3ad978982 Escape $ when wrapping Fixes #52640 2018-06-22 14:57:03 -07:00
Joao Moreno 28e2130870 tree with iterators 2018-06-22 22:52:50 +02:00
Joao Moreno a25a9b8083 iterator filter 2018-06-22 22:52:35 +02:00
Rob Lourens 8be20697c0 Settings editor - use inline checkbox control 2018-06-22 13:46:16 -07:00
Rob Lourens e0fbb4294b Change custom Checkbox to expose an onChange Event instead of taking an onChange callback in options.
Otherwise it's impossible to reference the checkbox from inside the callback while constructing a Checkbox
2018-06-22 12:54:25 -07:00
Rob Lourens bf4f7de576 Settings editor - move checkbox next to description 2018-06-22 12:54:25 -07:00
Rob Lourens dcf85b665f Settings editor - larger space between description and value 2018-06-22 12:54:25 -07:00
Joao Moreno 94343c80ef use iterators in tree model 2018-06-22 21:22:47 +02:00
Ramya Achutha Rao 51c1f85fab Revert expanding only non empty views as that would override user choice #52442 2018-06-22 11:53:20 -07:00
Ramya Achutha Rao eae9075aba Avoid Installed in default to avoid duplication #52442 2018-06-22 11:31:37 -07:00
Rob Lourens b15c521c74 Settings editor - tweak padding and spacing 2018-06-22 11:13:01 -07:00
Jackson Kearl 19c390e647 Fix error messag bug and imporve test case (#52698) 2018-06-22 10:40:35 -07:00
isidor 65dd9c0548 fixes #52192 2018-06-22 17:37:49 +02:00