Use update flutter app icon (#510) by maksnevor · Pull Request #537 · team113/messenger

maksnevor · 2023-08-31T10:59:28Z

Resolves #510

Synopsis

Неудобно обновлять иконки, нужно не забыть обновить их для каждой платформы в директориях android, ios, windows, macos, linux, web.

Solution

Использовать пакет update-flutter-app-icon, к нему написать update_launcher.json, который будет обновлять иконки во всем приложении.

Checklist

maksnevor · 2023-09-04T05:37:53Z

FCM

Update launcher icons with update-flutter-app-icon (#537, #510)

CONTRIBUTING.md

Makefile

update_icons.json

snap/gui/app_icon.desktop

assets/icons/launcher/android_launcher.png

assets/icons/launcher/ios_launcher.png

…her-icon-to-generate-icons

SleepySquash

@maksnevor, отписал кое-где просьбы поправить, кое-где сомнения.

SleepySquash · 2023-09-05T08:19:27Z

assets/icons/launcher/apple.png

@maksnevor, а почему appleа два? apple.png и apple_rounded? Если один для iOS, а второй для macOS, то какой смысл тогда их называть apple?

apple - для ios,android. apple_rounded - используется для macos форматов @2x@2x.

SleepySquash · 2023-09-05T08:22:21Z

assets/icons/launcher/android.png

@maksnevor, вообще смотрю и думаю об этом всём: чтобы сделать хорошую иконку, нужно так или иначе открывать фотошоп и нарезать эту иконку. Существуют всякие веб сервисы, которые генерируют иконки (они сразу генерируют пачку). Какой алгоритм действий должен быть, чтобы обновить иконки? Открыть фотошоп и нарезать 8-10 картинок?

Существуют, но генерируют они плохие пачки. Теряем много адаптации и форматов.

@maksnevor, https://icon.kitchen/ генерит хорошие пачки.

SleepySquash · 2023-09-05T08:23:23Z

Makefile

+icon.update:
+ifeq ($(dockerized),yes)
+	docker run --rm -v "$(PWD)":/app -w /app \
+	           -v "$(HOME)/.pub-cache":/usr/local/flutter/.pub-cache \


А зачем у этого конейтера маппить .pub-cache? Это же нода, она не использует .pub-cache.

SleepySquash · 2023-09-05T08:23:57Z

Makefile

+ifeq ($(dockerized),yes)
+	docker run --rm -v "$(PWD)":/app -w /app \
+	           -v "$(HOME)/.pub-cache":/usr/local/flutter/.pub-cache \
+		node:${NODE_VER} \


А NODE_VER - это что? Оно где-то определено?

В Makefile'е везде сейчас используется формат записи переменных среды в виде $( ) - с круглыми скобками. Давайте будем консистентны и придержимся единого формата.

SleepySquash · 2023-09-05T08:34:31Z

macos/Runner/Assets.xcassets/AppIcon.appiconset/app_icon_256.png

А почему иконка 256х256 квардаратная, а, например, 16@2x@2x.png - скруглённая с отступами?

Это же разные размеры одной и той же иконки?

А почему @2x@2x?

SleepySquash · 2023-09-05T08:49:14Z

CONTRIBUTING.md

+make icon.update dockerized=yes  # Docker-wrapped
+```
+
+__Note:__ To update, you need to download new icons along the path `assets/icons/launcher` and match the correct naming:


Иконки же не скачиваются. Их скорее размещают, т.к. где-то они уже лежат. И помещают скорее изображения, из которых будут нерезаны иконки.

SleepySquash · 2023-09-05T08:57:48Z

CONTRIBUTING.md

+* `linux.png` - for linux launcher icon
+* `web_alert.png` - for web launcher icon (alert)
+* `web.png` - for web launcher icon and favicon
+* `windows.png` - for windows launcher icon


Я не понял, почему иконка 256 на 256 сначала, а потом, прочитав инструкцию от Microsoft, понял, что на винде иконка 256 - это максимально оптимальных размер для иконки.

У меня в голове пока 2 сценария:

Если делаем для каждой ОС свои, то нужно описать требования к изображениям, которые туда кладутся.

Делаем одну 1024 на 1024 или SVG картиночку (возможно, две - бэкграунд и форграунд, т.к. андойд так требует), из которой как-то самостоятельно нарезаются иконки под все ОС, потому что все эти требования прочитать и учесть - тоже не очень приятная работа.

SleepySquash · 2023-09-05T09:15:49Z

Makefile

При выполнении пишет такое:

mayday@MBP messenger % make icon.update npx update-flutter-app-icon dev/update_icons.json Need to install the following packages: update-flutter-app-icon@0.2.0 Ok to proceed? (y)

Т.е. это надо устанавливать? А куда? Не лучше тогда в npm i --dev update-flutter-app-icon какой-нибудь делать, чтобы через npm вызывать?

SleepySquash · 2023-09-05T09:33:32Z

macos/Runner/Assets.xcassets/AppIcon.appiconset/app_icon_1024.png

Не app_icon, а icon должен быть префикс у файлов. .appiconset же имеет файл Contents.json, там написаны префиксы всех иконок. Аналогично и с iOS.

SleepySquash · 2023-09-05T09:38:31Z

Makefile

При выполнении мне пишет вот такое что-то (иконка там есть, но не обновляется):

=== windows === /Users/mayday/Developer/Flutter/messenger/windows/runner/resources/app_icon.ico [{"size":-1}] Error converting file "/Users/mayday/Developer/Flutter/messenger/windows/runner/resources/app_icon.ico": RangeError [ERR_OUT_OF_RANGE]: The value of "value" is out of range. It must be >= 0 and <= 255. Received 319

Use update flutter app icon

11e9304

maksnevor added enhancement Improvement of existing features or bugfix k::toolchain Changes of project toolchain labels Aug 31, 2023

maksnevor added this to the 0.1.0-alpha.10 milestone Aug 31, 2023

maksnevor self-assigned this Aug 31, 2023

maksnevor added 2 commits August 31, 2023 14:06

Fix generate for android

ab1241d

update contributing and makefile

2c9446f

maksnevor marked this pull request as ready for review September 4, 2023 05:38

maksnevor requested a review from SleepySquash September 4, 2023 05:38

SleepySquash requested changes Sep 4, 2023

View reviewed changes

maksnevor added 2 commits September 4, 2023 14:40

fix issues

8785a55

fix of the generation location

a75a63c

maksnevor requested a review from SleepySquash September 4, 2023 13:29

SleepySquash added 2 commits September 5, 2023 12:52

Corrections

8badad0

Merge remote-tracking branch 'origin/main' into 510-use-flutter-launc…

68e1553

…her-icon-to-generate-icons

SleepySquash requested changes Sep 5, 2023

View reviewed changes

SleepySquash assigned SleepySquash and unassigned maksnevor Sep 5, 2023

SleepySquash marked this pull request as draft September 5, 2023 15:58

SleepySquash modified the milestones: 0.1.0-alpha.10, 0.1.0-alpha.11 Oct 10, 2023

SleepySquash modified the milestones: 0.1.0-alpha.10.1, 0.1.0-alpha.11 Oct 17, 2023

SleepySquash modified the milestones: 0.1.0-alpha.11, 0.1.0-alpha.12 Nov 2, 2023

SleepySquash added the p::minor Issues/PRs which are not required to be resolved at the moment (temporary) label Nov 27, 2023

SleepySquash modified the milestones: 0.1.0-alpha.11.1, 0.1.0-alpha.12 Dec 22, 2023

SleepySquash removed this from the 0.1.0-alpha.12 milestone Feb 8, 2024

SleepySquash modified the milestones: 0.6.9, 0.7.0, 0.6.10 Nov 11, 2025

SleepySquash modified the milestones: 0.6.11, 0.7.0, 0.6.12 Nov 24, 2025

SleepySquash modified the milestones: 0.6.13, 0.7.0 Dec 8, 2025

SleepySquash modified the milestones: 0.6.14, 0.7.0, 0.8.0 Dec 19, 2025

SleepySquash modified the milestones: 0.7.1, 0.8.0, 0.7.2 Jan 7, 2026

SleepySquash modified the milestones: 0.8.0, 0.9.0, 0.8.1 Jan 21, 2026

SleepySquash modified the milestones: 0.8.2, 0.9.0, 0.10.0 Feb 2, 2026

SleepySquash modified the milestones: 0.9.1, 0.10.0, 0.9.2 Feb 11, 2026

Conversation

maksnevor commented Aug 31, 2023 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Synopsis

Solution

Checklist

Uh oh!

maksnevor commented Sep 4, 2023

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

SleepySquash left a comment

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

Comments

maksnevor commented Aug 31, 2023 •

edited

Loading