Skip to content

Enable usage of svg favicons and remove imagick from iconbuilder #36607

Open

Description

@szaimen

This allows us to get rid of using imagick in iconbuilder of the theming app, which in return allows us to not enforce this requirement anymore since gd and imaginary can be used for all other tasks.

Julius found this: https://www.leereamsnyder.com/blog/favicons-in-2021#safari-does-not-fully-support-svg-icons

cc @juliushaertl

Activity

  1. added this to the Nextcloud 27 milestone on Feb 8, 2023
  2. szaimen commented on Feb 8, 2023

    @szaimen
    ContributorAuthor

    also cc @nextcloud/ios

  3. juliusknorr commented on Feb 8, 2023

    @juliusknorr
    Member

    Seems to work just fine with an svg and the mask for safari:

    Screenshot 2023-02-08 at 14 10 10

    Now we'd need to check if there is a way to invert the svg to be filled with masking out the app icon or if the new safari icon design would be fine, but otherwise I'd say it would be a good step to get rid of imagick there.

  4. juliusknorr commented on Feb 8, 2023

    @juliusknorr
    Member

    Would also solve #27502

  5. szaimen commented on Feb 8, 2023

    @szaimen
    ContributorAuthor

    Great 🎉

  6. changed the title [-]investigate if safari now supports svg favicons [/-] [+]Enabke usage of svg favicons and remove imagick from iconbuilder[/+] on Feb 8, 2023
  7. changed the title [-]Enabke usage of svg favicons and remove imagick from iconbuilder[/-] [+]Enable usage of svg favicons and remove imagick from iconbuilder[/+] on Feb 8, 2023
  8. szaimen commented on Feb 8, 2023

    @szaimen
    ContributorAuthor

    Maybe someone with a mac could look into this for 27? :)

    I could help testing a PR on other platforms :)

  9. added
    1. to developAccepted and waiting to be taken care of
    and removed
    0. Needs triagePending check for reproducibility or if it fits our roadmap
    on Feb 8, 2023
  10. szaimen commented on Mar 7, 2023

    @szaimen
    ContributorAuthor
  11. AndyScherzinger commented on Mar 7, 2023

    @AndyScherzinger
    Member

    I am not having a mac, so If it works for Julius I think we are fine.

  12. removed this from the Nextcloud 27 milestone on May 1, 2023
  13. szaimen commented on May 1, 2023

    @szaimen
    ContributorAuthor

    Also cc @nextcloud/server

  14. solracsf commented on Sep 27, 2023

    @solracsf
    Member

    Any chances to see this landing in v28?

  15. szaimen commented on Sep 27, 2023

    @szaimen
    ContributorAuthor

    It was not prioritized so unfortunately no. But cc @sorbaugh and @AndyScherzinger

  16. AndyScherzinger commented on Sep 27, 2023

    @AndyScherzinger
    Member

    What @szaimen said. So given our priorities for v28 we won't have the capacity. However if somebody would be willing to contribute an implementation than we are more than happy to review'n'merge it.

  17. added a commit that references this issue on Sep 23, 2026
    21e740f
  18. added and removed
    1. to developAccepted and waiting to be taken care of
    on Sep 23, 2026
  19. self-assigned this
    on Sep 23, 2026
  20. szaimen commented on Sep 23, 2026

    @szaimen
    ContributorAuthor

    WIP in #64703

  21. added 2 commits that reference this issue on Sep 24, 2026
    8281c20
    d9e2b04
  22. removed their assignment
    on Oct 9, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions