Skip to content

Add confirmation button to share input #7196

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
MorrisJobke merged 17 commits into from
Mar 21, 2018
Merged

Add confirmation button to share input #7196

Changes from 1 commit
Commits
Show all changes
17 commits
Select commit Hold shift + click to select a range
ce7775a
Replace information icon with confirmation button in share input
jancborchardt Nov 16, 2017
6fef01c
Adjust search term to test
danxuliu Mar 16, 2018
375eab9
Add tests for emails and circles already shared with
danxuliu Mar 16, 2018
8af9c55
Add tests for exact search results already shared with
danxuliu Mar 16, 2018
3980364
Add autocompletion tests for each type of share
danxuliu Mar 16, 2018
d606219
Extract code to get suggestions to its own method
danxuliu Mar 16, 2018
fcef15a
Move stub setup outside the test method
danxuliu Mar 19, 2018
ed1452d
Use "showTemporary" instead of explicitly hiding the notification
danxuliu Mar 19, 2018
1c44051
Show an error when getting the suggestions succeeds with failure content
danxuliu Mar 19, 2018
89b0e34
Extract code to filter suggestions to its own function
danxuliu Mar 19, 2018
5e2a8cc
Return also exact matches besides all suggestions
danxuliu Mar 19, 2018
9371b61
Add a share when clicking on the confirm button
danxuliu Mar 19, 2018
10a4f8e
Confirm a share also by pressing enter on the input field
danxuliu Mar 20, 2018
6eb5cc5
Reuse last suggestions if the same parameters are used
danxuliu Mar 20, 2018
9a0fbe3
Discard cached suggestions when adding a share
danxuliu Mar 21, 2018
a2c52cd
Extract code to restore the UI after confirming a share
danxuliu Mar 20, 2018
203bf51
Keep showing the working icon while there are pending operations
danxuliu Mar 20, 2018
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Discard cached suggestions when adding a share 
The suggestions depend on the results returned by the server, but also
on the sharees already shared with. Due to that adding a share changes
the suggestions, so now the cached suggestions are discarded when a
share is added.

Signed-off-by: Daniel Calviño Sánchez <[email protected]>
  • Loading branch information
@danxuliu
danxuliu committed Mar 21, 2018
commit 9a0fbe307d1eb8583117e5c38ec1f4ae2b5e0f9a
6 changes: 6 additions & 0 deletions core/js/sharedialogview.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -445,6 +445,9 @@
$confirm.addClass('hidden');

this.model.addShare(s.item.value, {success: function() {
// Adding a share changes the suggestions.
self._lastSuggestions = undefined;

$(e.target).val('')
.attr('disabled', false);
$loading.addClass('hidden')
Expand Down Expand Up @@ -525,6 +528,9 @@
$loading.removeClass('inlineblock');
$confirm.removeClass('hidden');

// Adding a share changes the suggestions.
self._lastSuggestions = undefined;

$shareWithField.val('');
$shareWithField.prop('disabled', false);
$shareWithField.focus();
Expand Down