Bug 419341

Summary:	"packages to remove" view could be improved
Product:	[Applications] Discover	Reporter:	Nicolas Fella <nicolas.fella>
Component:	discover	Assignee:	Dan Leinir Turthra Jensen <leinir>
Status:	RESOLVED FIXED
Severity:	normal	CC:	aleixpol, nate
Priority:	NOR
Version:	unspecified
Target Milestone:	---
Platform:	Other
OS:	Linux
Latest Commit:	https://invent.kde.org/plasma/discover/commit/c0372aec881050a4fae18d04699255ad4b653e6d	Version Fixed In:	5.20
Attachments:	Screenshot

Description Nicolas Fella 2020-03-28 16:43:57 UTC

SUMMARY

I got this view (see attachment) and thought all those packages are going to be removed because I didn't see the "in order to install" at first. It would help to make this part of the message more prominent.

SOFTWARE/OS VERSIONS
KDE Plasma Version: master
KDE Frameworks Version: master
Qt Version: 5.14

Comment 1 Nicolas Fella 2020-03-28 16:44:38 UTC

Created attachment 127066 [details]
Screenshot

Comment 2 Aleix Pol 2020-07-30 23:21:54 UTC

This was already improved some time ago adding a "more" button IIRC. Can you confirm you still get this?

Comment 3 Nate Graham 2020-08-26 14:28:59 UTC

Git commit c0372aec881050a4fae18d04699255ad4b653e6d by Nate Graham.
Committed on 26/08/2020 at 14:27.
Pushed by ngraham into branch 'master'.

Improve UI of proceed sheet

Right now if the text is really long, it gets elided at the bottom so
you need to click a button to see the rest, and the two distinct
sections of the text are kind of unclear.

This commit puts the text in the sheet's own scrollview so you can scroll
immediately, and adds a line break between the sections. It also puts the
bottom buttons in a footer, which looks better.
FIXED-IN: 5.20

M  +33   -40   discover/qml/DiscoverWindow.qml
M  +1    -1    libdiscover/backends/PackageKitBackend/PackageKitUpdater.cpp

https://invent.kde.org/plasma/discover/commit/c0372aec881050a4fae18d04699255ad4b653e6d