31 Mar 2024 |
Maximiliano 🥑 | 👌 | 21:21:02 |
Maximiliano 🥑 | pushed to the MR | 21:21:48 |
Gitlab Notifications | [World/Shortwave] msandova
commented on
merge request !485: metainfo: Add app branding
The second commit is missing captions on the screenshots. Imo we need new ones (2 or 3 max) which represented different views.
| 21:23:14 |
Felix | Maximiliano 🥑: ty, will have a look soonish ™️ | 21:26:22 |
Maximiliano 🥑 | why did you change | 21:26:35 |
Maximiliano 🥑 | -<component type="desktop-application">
+<component type="desktop">
| 21:26:47 |
Felix | IIRC it's not a thing anymore in Appstream 1.0 | 21:29:50 |
Maximiliano 🥑 | https://www.freedesktop.org/software/appstream/docs/sect-Metadata-Application.html | 21:32:59 |
Maximiliano 🥑 | Download image.png | 21:33:11 |
Maximiliano 🥑 | TIL | 21:33:26 |
Felix | In reply to @msandova:gnome.org sandte ein Bild. Ah hmmm, then I mixed it up :D | 21:34:10 |
Maximiliano 🥑 | I would not blame anyone for being confused at the ever moving, ill-documented appstream documentation | 21:34:36 |
Maximiliano 🥑 | * I would not blame anyone for mixing up the ever moving, ill-documented appstream documentation | 21:34:51 |
Maximiliano 🥑 | * I would not blame anyone for mixing up the ever-moving, ill-documented appstream documentation | 21:34:56 |
Felix | Which is why I want to keep everything in sync / well structured | 21:36:46 |
Felix | I lost the overview before the overhaul with all those random MRs and what's best practices and what not | 21:37:30 |
Maximiliano 🥑 | e.g. https://github.com/ximion/appstream/issues/617 docs contradict checker | 21:37:41 |
Felix | Some of the keys were only present in one project but not in the other | 21:37:45 |
Felix | In reply to @msandova:gnome.org e.g. https://github.com/ximion/appstream/issues/617 docs contradict checker Uh oh. At least my id is all lower case | 21:38:31 |
Gitlab Notifications | [World/Shortwave] bertob
commented on
merge request !485: metainfo: Add app branding
LGTM
| 21:58:59 |
Maximiliano 🥑 | In reply to @felix:haecker.io Uh oh. At least my id is all lower case yeah, it does not really affect any of our apps. Its just one of those appstreamcli moments | 21:59:39 |
1 Apr 2024 |
Gitlab Notifications | [World/Shortwave] janvlug
commented on
issue #718: With buffering issues, Shortwave can stop the play and never resume it
I encounter this issue very often on my Librem 5 Linux phone. It happens especially when I play a radio station when on WiFi, and when the WiFi gets out of reach the phone switches to 4G internet. The only way then to start playing a radio station again is by totally closing the Shortwave app and starting it again.
| 11:15:43 |
Gitlab Notifications | [World/Shortwave] janvlug
opened issue #725: Improve controls on lock screen
I use Shortwave on a Librem 5 Linux phone.
When I start playing a station, and lock the screen, the station keeps playing. Great.
On the lock screen I see controls for the station I'm playing:
Here I have some requests and remarks:
- I can touch the pause button in the middle. This stops the playing of the station. However, it also removes the controls from the lock screen, so I cannot start playing the station without unlocking the screen. This is inconvenient. Please do not remove the controls when stopping playing the station.
- I think that the symbol for stopping the station should be a rectangle, and not the pause symbol, because you cannot really pause playing a station. You can stop and start again however.
- The other buttons (back to begin, 10 sec back, 30 sec forward, forward to next) do not work, obviously, because you cannot do this on a radio station. These buttons do not do anything, so the should all be displayed disabled (or maybe totally removed, if possible).
| 11:50:40 |
Gitlab Notifications | [World/Shortwave] haecker-felix
commented on
merge request !485: metainfo: Add app branding
I think the size information isn't needed anymore and can be dropped
| 14:10:35 |
Gitlab Notifications | [World/Shortwave] msandova
commented on
merge request !485: metainfo: Add app branding
Indeed. done.
| 14:31:07 |
Gitlab Notifications | [World/Shortwave] msandova
commented on
merge request !485: metainfo: Add app branding
For context, this info does nothing besides annoying people when they update the screenshots and forgot to update these numbers.
| 15:23:24 |
Gitlab Notifications | [World/Shortwave] haecker-felix
pushed
1 commit
to
main:
| 20:28:43 |
Gitlab Notifications | [World/Shortwave] haecker-felix
pushed
1 commit
to
main:
-
306d88f7
meson: Allow vcs_tag command to fail
| 20:38:01 |
2 Apr 2024 |
Gitlab Notifications | [World/Shortwave] haecker-felix
commented on
merge request !485: metainfo: Add app branding
developer_name is deprecated and got replaced by developer
| 09:37:40 |
Gitlab Notifications | [World/Shortwave] msandova
commented on
merge request !485: metainfo: Add app branding
Yes, but this will make any frontend not supporting appstreamcli 1.0 fail to find an author. I dont think there many front ends supporting 1.0.
| 09:44:35 |