data/icons: update icons #493
Dismiss Review
Are you sure you want to dismiss this review?
Labels
Clear labels
AI
Eolite
FS
Good First PR
GSoC
HardwareTested
HLL
Influence/Settings
Influence/Text/TYPO
IRCC
Lang/C
Lang/FASM
Pay for the code
Subsystem/API
Subsystem/Audio
Subsystem/Graphics
Subsystem/IPC and events
Subsystem/Memory
Subsystem/Network
Subsystem/Services(daemon)
Subsystem/Taskmanager
Subsystem/VFS
Subsystem/Window
Category
Applications
Category
Drivers
Category
General
Category
Kernel
Category
Libraries
The issue is suitable to beginners
This issue or PR in the Google Source of Code program
Kind
Breaking
Breaking change that won't be backward compatible
Kind
Bug
Something is not working
Kind
Build
Kind
Documentation
Documentation changes
Kind
Enhancement
Improve existing functionality
Kind
Feature
New functionality
Kind
Security
This is security issue
Kind
Testing
Issue or pull request related to testing
Paid task
PR
Conflicts
PR conflicts with main
PR
Dependent
This PR is dependent on another PR
Priority
Critical
1
The priority is critical
Priority
High
2
The priority is high
Priority
Low
4
The priority is low
Priority
Medium
3
The priority is medium
PR
Ready to merge
Pull request is ready for merge
PR
Request changes
Changes requested in pull request
PR
Review required
Reviewed
Confirmed
Issue has been confirmed
Reviewed
Duplicate
This issue or pull request already exists
Reviewed
Invalid
Invalid issue
Reviewed
Won't Fix
This issue won't be fixed
Status
Abandoned
Somebody has started to work on this but abandoned work
Status
Blocked
Something is blocking this issue or pull request
Status
Need More Info
Feedback is required to reproduce issue or to continue work
infinity service, audio drivers, midi, speacker, audio programs
vesa, vga, framebuffer, cursors, blitter, and video drivers
pipes, signals, events, shared memory
virt and phys memory allocators, malloc and other
userspace and kernel(for example: serial) services
process, threads, run apps, scheduler
drivers from filesystem, fs api, blkdev, programs that work with the file system
windows, skins, buttons, mouse and keyboard code for windows (not the base code)
No labels
Milestone
No items
No Milestone
Projects
Clear projects
No projects
No Assignees
Notifications
Due Date
No due date set.
Blocks
#507 WIP: apps/webview: update 4.0
KolibriOS/kolibrios
Reference: KolibriOS/kolibrios#493
Reference in New Issue
Block a user
Blocking a user prevents them from interacting with repositories, such as opening or commenting on pull requests or issues. Learn more about blocking a user.
ICONS18:
ICONS32:
Other:
.ob7files inmenu.datJust merge, please.
Please could you note down visual changes this introduces?Thanks for updating the main comment! 🙏Icons updateto data/icons: update iconsWhy is the WebView icon changed to a Light house?... to look like NetscapeNavigator...? 👀
The design seems a bit out of place and a little 'retro'...? I just want to check that it is deliberate.
а куда гриб делся?

и для чего новая иконка вместо иконки файла fb2?
UPD: иконка информации(последняя в старом варианте) в прежней версии выглядит лучше
Request to return "Info" icon which can be useful for many programs.
гриба больше не будет, он плохо выглядел
Оставьте иконки дизайнеру. Просто смержите.
@ace-dent
WebView icon change is fully intentional - it's preparation for #507.
@Doczom
Вместо гриба теперь новая иконка.
cause Leency
Hi @Leency - please don't be insulted... it's actually a compliment to how excellent your icons have been!
(The outstanding design was actually what drew me into this project; few Hobby OS have such good looking desktops!!)
WebView is your program; of course it can look however you see fit!
Personally I think the lighthouse is an outdated reference; the symbology of a window on the world is much better. The actual pixel art doesn't match the aesthetic of your other bitmap designs; your other artwork has softer gradients. But it's your choice. 🙇🏼♂️
Again- forgive my critique. I love your design and artwork that has truly elevated this OS.
Technical question:
@Burer @Leency - Is there a reason to make these 8 bits per channel (24 / 32 bpp)? It's possible indexed (/ paletted) formats are more compact. I'm happy to optimise the final design once the PR is merged- or can recommend using OxiPNG.
@ace-dent
It is possible, but it will require rewriting
reshareto read 8-bit and publish 32-bit images (because all programs, that usereshare, expect 32-bit images), or to also rewrite all programs usingreshareto work with 8-bit images.Thus, that is a topic for another PR.
Current version is compressed via optipng, which gives good (not sure if best) resuts.
No problem, thanks a lot for a sweet words! <3
I just like the lighthouse icon... I feel it more alive and fresh than a very common Earth symbol :)