[Discover] [Bug 390148] No feedback when I try to add an invalid source URI/URL

[Nate Graham]

https://bugs.kde.org/show_bug.cgi?id=390148

Nate Graham  changed:

   What|Removed |Added

 Status|REOPENED|RESOLVED
  Latest Commit|https://commits.kde.org/dis |https://commits.kde.org/dis
   |cover/2b9e1b5e1ebbfab384b3f |cover/b623a4afb12f0bfa74f76
   |e3febe63e0ef076c273 |711598e84772a1c6d2f
 Resolution|--- |FIXED

--- Comment #11 from Nate Graham  ---
Git commit b623a4afb12f0bfa74f76711598e84772a1c6d2f by Nathaniel Graham.
Committed on 06/03/2018 at 12:28.
Pushed by ngraham into branch 'master'.

Fix the Add Source dialog

Summary:
Fix a variety of bugs and usability issues with the {nav Add Source} dialog:
- Make the close button actually close the dialog
- Give the text field focus so you can immediately start typing
- Make the return key push the Add button
- Remove needless close symbol in top-right corner
- Give the buttons icons
- Re-word the title and make it larger

Test Plan:
Before:
{F5738465}

After:
{F5738429}

Tested with Flatpak backend:
- Invalid URLs are rejected
- Valid URLs that nonetheless don't point to a Flatpak repo trigger an error
message
- The close button now closes the dialog
- The escape key still closes the dialog
- The return and enter keys press the {nav Add} button

Reviewers: #discover_software_store, apol

Reviewed By: #discover_software_store, apol

Subscribers: acrouthamel, plasma-devel

Tags: #plasma

Differential Revision: https://phabricator.kde.org/D11003

M  +22   -20   discover/qml/AddSourceDialog.qml

https://commits.kde.org/discover/b623a4afb12f0bfa74f76711598e84772a1c6d2f

-- 
You are receiving this mail because:
You are watching all bug changes.

[Discover] [Bug 390148] No feedback when I try to add an invalid source URI/URL

[Nate Graham]

https://bugs.kde.org/show_bug.cgi?id=390148

--- Comment #10 from Nate Graham  ---
I've submitted a patch that fixes all of the above-mentioned issues:
https://phabricator.kde.org/D11003

-- 
You are receiving this mail because:
You are watching all bug changes.

[Discover] [Bug 390148] No feedback when I try to add an invalid source URI/URL

[Nate Graham]

https://bugs.kde.org/show_bug.cgi?id=390148

--- Comment #9 from Nate Graham  ---
Created attachment 17
  --> https://bugs.kde.org/attachment.cgi?id=17=edit
What the "Add source" dialog looks like now

Sure, here's what it looks like now as of your latest commit.

The red symbolic close button that you just added works, but shouldn't even be
there at all since there's a textual Close button below. Alas, the textual
close button doesn't work; clicking on it does nothing.

Also, the "close" button should probably be renamed "cancel", neither button
has an icon, and the return key doesn't invoke the OK button. All in all
there's a lot of work to be done here, but we can track those with other bugs
if you'd like.

-- 
You are receiving this mail because:
You are watching all bug changes.

[Discover] [Bug 390148] No feedback when I try to add an invalid source URI/URL

[Aleix Pol]

https://bugs.kde.org/show_bug.cgi?id=390148

--- Comment #8 from Aleix Pol  ---
Can you show me what you see? I have no idea what you are talking about.

-- 
You are receiving this mail because:
You are watching all bug changes.

[Discover] [Bug 390148] No feedback when I try to add an invalid source URI/URL

[Nate Graham]

https://bugs.kde.org/show_bug.cgi?id=390148

Nate Graham  changed:

   What|Removed |Added

 Status|RESOLVED|REOPENED
 Resolution|FIXED   |---

--- Comment #7 from Nate Graham  ---
I don't think that's the right fix. That commit adds a symbolic close button in
the corner that does work (though it's blurry and pixellated for some reason),
but the textual close button still does nothing. Let's fix the existing close
button rather than leaving it broken and adding a redundant control.

-- 
You are receiving this mail because:
You are watching all bug changes.

[Discover] [Bug 390148] No feedback when I try to add an invalid source URI/URL

[Aleix Pol]

https://bugs.kde.org/show_bug.cgi?id=390148

Aleix Pol  changed:

   What|Removed |Added

 Resolution|--- |FIXED
  Latest Commit|https://commits.kde.org/dis |https://commits.kde.org/dis
   |cover/c9bff67739e672454ec73 |cover/2b9e1b5e1ebbfab384b3f
   |3377a64b0518c46fbdc |e3febe63e0ef076c273
 Status|REOPENED|RESOLVED

--- Comment #6 from Aleix Pol  ---
Git commit 2b9e1b5e1ebbfab384b3fe3febe63e0ef076c273 by Aleix Pol.
Committed on 01/03/2018 at 12:15.
Pushed by apol into branch 'Plasma/5.12'.

Add a close button to AddSourceDialog

M  +12   -0discover/qml/AddSourceDialog.qml

https://commits.kde.org/discover/2b9e1b5e1ebbfab384b3fe3febe63e0ef076c273

-- 
You are receiving this mail because:
You are watching all bug changes.

[Discover] [Bug 390148] No feedback when I try to add an invalid source URI/URL

[Nate Graham]

https://bugs.kde.org/show_bug.cgi?id=390148

Nate Graham  changed:

   What|Removed |Added

 Status|RESOLVED|REOPENED
 Resolution|FIXED   |---
 Ever confirmed|0   |1

--- Comment #5 from Nate Graham  ---
This commit introduces a fairly significant usability regression: the pop-up's
close button is non-functional, so there's no visible way to get rid of it!

-- 
You are receiving this mail because:
You are watching all bug changes.

[Discover] [Bug 390148] No feedback when I try to add an invalid source URI/URL

[Aleix Pol]

https://bugs.kde.org/show_bug.cgi?id=390148

Aleix Pol  changed:

   What|Removed |Added

 Status|UNCONFIRMED |RESOLVED
 Resolution|--- |FIXED
  Latest Commit||https://commits.kde.org/dis
   ||cover/c9bff67739e672454ec73
   ||3377a64b0518c46fbdc

--- Comment #4 from Aleix Pol  ---
Git commit c9bff67739e672454ec733377a64b0518c46fbdc by Aleix Pol.
Committed on 26/02/2018 at 15:43.
Pushed by apol into branch 'Plasma/5.12'.

Don't close the add source dialog until a proper source has been offered

M  +21   -6discover/qml/AddSourceDialog.qml
M  +5-0libdiscover/backends/DummyBackend/DummyBackend.cpp
M  +1-0libdiscover/backends/DummyBackend/DummyBackend.h
M  +3-0libdiscover/backends/DummyBackend/DummySourcesBackend.cpp

https://commits.kde.org/discover/c9bff67739e672454ec733377a64b0518c46fbdc

-- 
You are receiving this mail because:
You are watching all bug changes.

[Discover] [Bug 390148] No feedback when I try to add an invalid source URI/URL

[Nate Graham]

https://bugs.kde.org/show_bug.cgi?id=390148

Nate Graham  changed:

   What|Removed |Added

   Keywords||usability

-- 
You are receiving this mail because:
You are watching all bug changes.

[Discover] [Bug 390148] No feedback when I try to add an invalid source URI/URL

[Dr . Chapatin]

https://bugs.kde.org/show_bug.cgi?id=390148

--- Comment #3 from Dr. Chapatin  ---
(In reply to Nate Graham from comment #2)
> This is in the dialog that appears from Settings > hamburger dropdown menu
> for a backend > Add repo/source.

Exactly.

-- 
You are receiving this mail because:
You are watching all bug changes.

[Discover] [Bug 390148] No feedback when I try to add an invalid source URI/URL

[Nate Graham]

https://bugs.kde.org/show_bug.cgi?id=390148

--- Comment #2 from Nate Graham  ---
This is in the dialog that appears from Settings > hamburger dropdown menu for
a backend > Add repo/source.

-- 
You are receiving this mail because:
You are watching all bug changes.

[Discover] [Bug 390148] No feedback when I try to add an invalid source URI/URL

[Aleix Pol]

https://bugs.kde.org/show_bug.cgi?id=390148

--- Comment #1 from Aleix Pol  ---
Where do you add it?

-- 
You are receiving this mail because:
You are watching all bug changes.

[Discover] [Bug 390148] No feedback when I try to add an invalid source URI/URL

[Nate Graham]

https://bugs.kde.org/show_bug.cgi?id=390148

Nate Graham  changed:

   What|Removed |Added

 CC||pointedst...@zoho.com

-- 
You are receiving this mail because:
You are watching all bug changes.