Minor UI issues
Having tested the latest code, some minor UI issues I observed:
- There's no generic "Open..." menu item. Presumably that's something we want?
- The "Set as Wallpaper" menu item should be labelled "Set as Background..." (background is the term used in the portal, the shell, Settings; the ellipsis is needed to indicate that further input is required before the action will be carried out)
- If I choose "Set as Wallpaper", and then click cancel in the portal dialog, I get a toast with the label "Could not set wallpaper." In this situation, I think that it would be better to silently fail. However, if the toast is required, "Could not set background" would be a better label (note the lack of period).
I've updated the mockups for some of this. See: