ngraham added a comment.

  Give the scrollview a frame and a white background or else apps look cut off 
and weird and there's a mysteriously hovering scroller: F7616694: 
Screenshot_20191017_111833.png <https://phabricator.kde.org/F7616694>
  
  Also why are there duplicates in here?
  
  Finally, I would consider adding the "Other applications are available in 
Discover" text to the bottom, so it doesn't interrupt the dialog's flow.

INLINE COMMENTS

> AppChooserDialog.qml:122
> +
> +            MouseArea {
> +                id: mouseArea

Change the cursor to the pointing hand cursor when hovering over something that 
will immediately perform an action when clicked

> AppChooserDialog.qml:140
> +
> +                Label {
> +                    anchors {

Allow this label to become a multi-line string for apps that have long-ish 
names:
F7616689: Screenshot_20191017_111655.png <https://phabricator.kde.org/F7616689>

REPOSITORY
  R838 Flatpak Support: KDE Portal for XDG Desktop

REVISION DETAIL
  https://phabricator.kde.org/D24732

To: jgrulich, #plasma
Cc: GB_2, ngraham, apol, ognarb, plasma-devel, LeGast00n, The-Feren-OS-Dev, 
jraleigh, fbampaloukas, ragreen, ZrenBot, alexeymin, himcesjf, lesliezhai, 
ali-mohamed, jensreuterberg, abetts, sebas, ahiemstra, mart

Reply via email to