Bug 390148

Where do you add it?

This is in the dialog that appears from Settings > hamburger dropdown menu for a backend > Add repo/source.

(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.

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   -6    discover/qml/AddSourceDialog.qml
M  +5    -0    libdiscover/backends/DummyBackend/DummyBackend.cpp
M  +1    -0    libdiscover/backends/DummyBackend/DummyBackend.h
M  +3    -0    libdiscover/backends/DummyBackend/DummySourcesBackend.cpp

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

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!

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   -0    discover/qml/AddSourceDialog.qml

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

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.

Can you show me what you see? I have no idea what you are talking about.

Created attachment 111117 [details]
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.

I've submitted a patch that fixes all of the above-mentioned issues: https://phabricator.kde.org/D11003

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