elementary / photos

Photo viewer and organizer designed for elementary OS
https://elementary.io
GNU Lesser General Public License v2.1
105 stars 39 forks source link

For elementary OSv8: support for png format #775

Open megatux opened 1 week ago

megatux commented 1 week ago

Problem

Tried the daily build of elementary OS 8. Took screenshots with included screenshot tool. Saved as png images. Tried to open from Files app. Expected to be able to see it with Photos but seems like it does not support png images?

Proposal

Screenshot tool should saved in a supported image format by Photos so no new software is needed.

Prior Art (Optional)

No response

jeremypw commented 1 week ago

Just tried this with some screenshots on OS8. I did not have any problem opening them from Files. I secondary-clicked on the icon and chose the "Open in Photo Viewer" option. it also worked by double-clicking the thumbnail in Files. Photos has always supported .png files afaik. Not sure what has happened in your case. Looks like the association between Photos and the png mimetype has been broken.

You could try using the "Open with ..." option in the context menu and choose "Photo Viewer" from there (or go in to "Other applications ..." if necessary and make sure "Set as default" is clicked).

megatux commented 1 week ago

Oh, I will retry later. Thanks!

megatux commented 6 days ago

Hi again, I retried with the daily build for days 09-08 and 09-13 . Photos and most other apps do not open (web browser, Music, Videos, etc). I could not diagnose the issue. This is in live USB OS try, not after installing it. Probably we should close this issue but I'd like to find a way to test the OS. Will ask for help in Discord. Thanks again

ryonakano commented 6 days ago

Photos and most other apps do not open (web browser, Music, Videos, etc). I could not diagnose the issue. This is in live USB OS try, not after installing it.

Hmm your issue sounds like https://github.com/elementary/triage/issues/215

jeremypw commented 6 days ago

FIles, Terminal and Code are not supplied in a flatpak yet, do they run?

megatux commented 5 days ago

Ok, now I'm writing this from Demo Run. I've found a workaround:

No Flatpak apps were running, only native debs. Running from terminal the flatpak run command failed with any app with this message:

$ flatpak run org.gnome.Evince 
bwrap: Creating new namespace failed: Permission denied
error: ldconfig failed, exit status 256

after some Googling I read that there were some issues with the file attributes of the bwrap binary and that it needs suid flag

ll  /usr/bin/bwrap
-rwxr-xr-x 1 root root 72160 mar 31 08:43 /usr/bin/bwrap*

$ sudo chmod u+s /usr/bin/bwrap 
$ ll  /usr/bin/bwrap
-rwsr-xr-x 1 root root 72160 mar 31 08:43 /usr/bin/bwrap*

$ flatpak run org.gnome.Evince 
Gtk-Message: 00:41:32.682: Failed to load module "canberra-gtk-module"
Gtk-Message: 00:41:32.691: Failed to load module "canberra-gtk-module"

and now any flatpak app run fine. Photos, Music, Web *I'm writing from Web now), etc

$ flatpak ps
Instancia  PID  Aplicación           Runtime
528172335  5929 org.gnome.Epiphany   io.elementary.Platform
357903542  6368 org.gnome.Epiphany   io.elementary.Platform
1131071555 6565 org.gnome.Epiphany   io.elementary.Platform
500435236  6845 org.gnome.Epiphany   io.elementary.Platform
2952605586 7052 org.gnome.Epiphany   io.elementary.Platform
2240082566 7254 org.gnome.Epiphany   io.elementary.Platform
1871649856 7989 io.elementary.music  io.elementary.Platform
1152536878 8033 io.elementary.videos io.elementary.Platform
4027606880 8130 io.elementary.screenshot io.elementary.Platform

So...not sure if this Flatpak issue should be move somewhere else?

BTW, Photos still was not the default app for png files, nor was listed unless you select Show all apps in the dialog.

zeebok commented 5 days ago

Photos not the default app is already covered in #769