GNOME Bugzilla – Bug 770860
Update screenshot in appdata
Last modified: 2017-09-29 13:41:35 UTC
The current screenshot in geary.appdata.xml points to Yorba and is very old. It needs to be updated.
In light of bug 775215, I would like to use this bug to encompass everything on updating our appdata.xml.in, since it is quite old already. * Update the screenshot (suggestions on what screenshots we could use can be found in bug 775215) * With GNOME software, users can read the release notes if they are included in the appdata.xml. An example can be found in GNOME Software's appdata [1] itself. If we want, we could write a script that still appends to NEWS for those not interested in reading the XML file. * Add a URL type="translate" for interested translators. For an example, see [2]. [2] https://github.com/GNOME/gnome-software/blob/master/data/appdata/org.gnome.Software.appdata.xml.in#L45 [3] https://github.com/GNOME/gnome-software/blob/master/data/appdata/org.gnome.Software.appdata.xml.in#L626
(In reply to Niels De Graef from comment #1) > [2] > https://github.com/GNOME/gnome-software/blob/master/data/appdata/org.gnome. > Software.appdata.xml.in#L45 > [3] > https://github.com/GNOME/gnome-software/blob/master/data/appdata/org.gnome. > Software.appdata.xml.in#L626 Whoops, those should be [1] and [2] respectively.
Now that UI changes are going to be pretty minimal for 0.12, this is probably worth using master as-is. Gautier, were you looking into this?
If Geary is ready for it, yeah I might give it a try.
But I think Geary is not ready yet Mike: for instance when I edit a draft it automatically closes itself after a few seconds (I think we are talk about it on Bug 777642: see your comment 6). Anyway, would be great to add a depends on bug 775215 here.
Oh, well there's a long list of bugs to fix yet, yeah, but the user interface shouldn't change itself. I was thinking that it's maybe worth putting versioned links into the appdata file for the screenshot, so that users always see an appropriate screenshot for the version they are about to install. That would also mean we can upload screenshots to the wiki whenever they ready and just update the appdata file on master, without giving anyone any false impressions.
Hey Gautier, I was wondering did you get to make any of these screenshots? I've got my phd out the way (for the moment) so would be keen to get them updated and get 0.12 out the door.
Good to see you again Michael :). I haven't made these because of some bugs in the mean time that prevented me from doing them - these days I'm not using Geary anymore so I won't be able to make them unfortunately.
Hey Gautier, oh no problem. If you catch me on IRC I'd be interested to know what lead you jump ship, don't feel obliged to though. Sorry to lose you, but thanks for your contributions!
Updated appdata to point to screenshots on the wiki in commit 8a20a45.
I've also added a translate URL to the app data as suggested above in commit d6ea9a1 I wouldn't mind including release notes in the appdata as well, but since these are already duplicated in the release email, debian/changelog, and ideally on a web site somewhere (although the archived release message might work for that), it would be good to find a solution that doesn't mean having to duplicate the release notes in 3 places instead of 2. It's maybe worth opening a new bug for that? The two screnshots I attached are pretty basic, but at least they show the 0.12 interface and the composer, which imho is a substantial improvement. The app data points to screenshots on wiki the at the moment so they aren't duplicated in git and are hosted somewhere. Lets continue to tune the screenshots as part of revamping the wiki over in Bug 775215.
About the redundancy: the `appstream-util` command provides both the appdata-to-news and news-to-appdata commands. That means that we can generate one from the other. Whether you think that's an acceptable compromise is up to you.