Board.KolibriOS.org

Official KolibriOS board
It is currently Sat Aug 17, 2019 8:34 pm

All times are UTC+03:00




Post new topic  Reply to topic  [ 100 posts ]  Go to page Previous 13 4 5 6 7 Next
Author Message
 Post subject: Re: @notify
PostPosted: Sun Oct 13, 2013 1:13 pm 
Offline
Mentor/Kernel Developer
User avatar

Joined: Fri Jun 30, 2006 9:01 am
Posts: 1247
eAndrew wrote:
As I said, I'm working on dynamic loading of icons in png format.
Sory for bad English


While it might sound interesting at first, I believe it is too much hassle for this project.
Think about loading times, code size, ram occupation,... all will be better without PNG. :)

_________________
"Any intelligent fool can make things bigger, more complex, and more violent. It takes a touch of genius -- and a lot of courage -- to move in the opposite direction." Albert Einstein


Top
   
 Post subject: Re: @notify
PostPosted: Sun Oct 13, 2013 1:20 pm 
Offline
User avatar

Joined: Sat Oct 05, 2013 9:32 pm
Posts: 386
Ok) But 7kb for simple notify program is very big size. May be dynamic loading raw icons? It is faster i think.


Top
   
 Post subject: Re: @notify
PostPosted: Sun Oct 13, 2013 1:52 pm 
Offline
Designer
User avatar

Joined: Thu Jan 25, 2007 3:33 pm
Posts: 5054
Программа всё дальше уходит от принципов KISS. Не нужно превращать такую простую программу в монстра. Если одному программисту захочется зелёный ободок, второму красный, третьему заленый, то будут выскакивать попугаи? Возможность менять цвет ободка лучше убрать.

Недочёты:
1. если провести другим окном над окном @notify, то у @notify остаются артефакты в уголках
Attachment:
Screenshot_1.png
Screenshot_1.png [ 4.26 KiB | Viewed 1816 times ]

2. 8 и далее поток отображается не ниже 7, а накладывается на 7
Attachment:
Screenshot_2.png
Screenshot_2.png [ 57.32 KiB | Viewed 1816 times ]

_________________
Через тернии к звездам


Top
   
 Post subject: Re: @notify
PostPosted: Sun Oct 13, 2013 2:03 pm 
Offline
User avatar

Joined: Sat Oct 05, 2013 9:32 pm
Posts: 386
Да, проблему с потоками я знаю. В чем проблема, не знаю. Уберу рамку фон и цвет текста. Обводку все таки оставлю, ибо она не сильно то и мешает. Или убрать. Или оставить.... Вообщем я подумаю. Насчет артефактов - спасибо, исправлю.
P.S.: что за принципы KISS?


Top
   
 Post subject: Re: @notify
PostPosted: Sun Oct 13, 2013 2:14 pm 
Offline
Kernel Developer

Joined: Sun Feb 10, 2013 12:37 pm
Posts: 2329
eAndrew wrote:
P.S.: что за принципы KISS?

http://ru.wikipedia.org/wiki/KISS_(принцип)

_________________
Всем чмоки в этом проекте! Засуньте эти 11 лет себе в жопу!


Top
   
 Post subject: Re: @notify
PostPosted: Sun Oct 13, 2013 2:15 pm 
Offline
Designer
User avatar

Joined: Thu Jan 25, 2007 3:33 pm
Posts: 5054
KISS (keep it short and simple) - процесс и принцип проектирования, при котором простота системы декларируется в качестве основной цели и/или ценности.
Т.е. должно быть сделано так максимально просто как это возможно, но не проще :)

_________________
Через тернии к звездам


Top
   
 Post subject: Re: @notify
PostPosted: Sun Oct 13, 2013 2:18 pm 
Offline
User avatar

Joined: Sat Oct 05, 2013 9:32 pm
Posts: 386
Спасибо, буду знать.
Если подумать, то я только убрал полупрозрачность (упростил) и добавил иконки, свечение, отмену автозакрытия. Вот вообщем и все.


Top
   
 Post subject: Re: @notify
PostPosted: Sun Oct 13, 2013 2:53 pm 
Offline

Joined: Sun Oct 30, 2011 6:43 pm
Posts: 1499
hidnplayr wrote:
Think about loading times, code size, ram occupation,... all will be better without PNG. :)
eAndrew wrote:
6. {B, C, E, I, M, N, W} - Иконки!!!
eAndrew wrote:
По умолчанию отсутствует.
If we don't want icons, then there is no need to load nor library, nor files in memory. I think png will be better in this case.


Top
   
 Post subject: Re: @notify
PostPosted: Sun Oct 13, 2013 6:52 pm 
Offline
Mentor/Kernel Developer
User avatar

Joined: Fri Jun 30, 2006 9:01 am
Posts: 1247
0CodErr wrote:
hidnplayr wrote:
Think about loading times, code size, ram occupation,... all will be better without PNG. :)
eAndrew wrote:
6. {B, C, E, I, M, N, W} - Иконки!!!
eAndrew wrote:
По умолчанию отсутствует.
If we don't want icons, then there is no need to load nor library, nor files in memory. I think png will be better in this case.


I would agree if the icons were 192x192 pixels, but not for such tiny ones...

_________________
"Any intelligent fool can make things bigger, more complex, and more violent. It takes a touch of genius -- and a lot of courage -- to move in the opposite direction." Albert Einstein


Top
   
 Post subject: Re: @notify
PostPosted: Sun Oct 13, 2013 7:06 pm 
Offline
Public Relations
User avatar

Joined: Mon Jun 07, 2010 12:01 pm
Posts: 1879
Leency wrote:
Программа всё дальше уходит от принципов KISS. Не нужно превращать такую простую программу в монстра. Если одному программисту захочется зелёный ободок, второму красный, третьему заленый, то будут выскакивать попугаи? Возможность менять цвет ободка лучше убрать.
Если проблема только в размере, а в скорости нет изменений, то очень неплохой функционал всего за 7Kb. Ты же сам добавлял вещи, которые функциональности не добавляют (как тени и полупрозрачность), и почему-то тогда это тебе не мешало? Лично я очень доволен результатом работы eAndrew, и хотел бы видеть последнюю версию 0.9 (без удаления уже сделанного, и с исправленными багами) в ночных сборках.

Вот баги нужно чинить, безусловно :-)


Top
   
 Post subject: Re: @notify
PostPosted: Sun Oct 13, 2013 8:23 pm 
Offline
User avatar

Joined: Sat Oct 05, 2013 9:32 pm
Posts: 386
Придется многое переписать.
Вообщем вот возможный примерный план работы @notify(вот он уже точно отходит от KISS, но его предложил Mario(вроде)).
Суть в чем. Где то в памяти висит процесс(что то вроде сервера, назовем notifyCenter). Окна он не имеет, но за то в нем все данные о нотифаях. Когда открывается окно нотифая, то отправляет свои данные(ширина высота, приоритет) в notifyCenter, которая расчитывает координаты для нового нотифая с учетом других нотифаев и отпровляет ей данные. Если нотифай удаляется, то notifyCenter опять же расчитывает координаты всех оставшихся нотифаев и отправляет им данные.
Для этого нужно использовать разшаривание памяти, что я пока что не умею, но это не проблема. Все это позволит добавить многострочность, автосмещение нотифаев вверх, если какое-то закрылось. Но это уже точно будет монстр.


Top
   
 Post subject: Re: @notify
PostPosted: Sun Oct 13, 2013 10:43 pm 
Offline
Kernel Developer

Joined: Sun Feb 10, 2013 12:37 pm
Posts: 2329
eAndrew wrote:
Но это уже точно будет монстр.

Монстр говоришь? Зацени код простенького OpenDialog.

_________________
Всем чмоки в этом проекте! Засуньте эти 11 лет себе в жопу!


Top
   
 Post subject: Re: @notify
PostPosted: Sun Oct 13, 2013 11:36 pm 
Offline
User avatar

Joined: Sat Oct 05, 2013 9:32 pm
Posts: 386
Я имею в виду монстр для уведомлений. А в код OpenDialog загляну)


Top
   
 Post subject: Re: @notify
PostPosted: Sun Oct 13, 2013 11:43 pm 
Offline
Public Relations
User avatar

Joined: Mon Jun 07, 2010 12:01 pm
Posts: 1879
eAndrew wrote:
Я имею в виду монстр для уведомлений. А в код OpenDialog загляну)
У тебя отличная версия уже сейчас - только баги починить. Убирать ничего не нужно :-)


Top
   
 Post subject: Re: @notify
PostPosted: Mon Oct 14, 2013 7:06 am 
Offline

Joined: Sun Oct 30, 2011 6:43 pm
Posts: 1499
eAndrew wrote:
Все это позволит добавить многострочность, автосмещение нотифаев вверх, если какое-то закрылось.
Quote:
многострочность
Можно сделать и без этого.
Quote:
автосмещение
Может добавить неудобства:
  • пользователь только захочет закрыть программу щелчком мыши по окну — а окно в это время перемещается
  • пользователь читает выведенное уведомление — окно начинает перемещаться


Top
   
Display posts from previous:  Sort by  
Post new topic  Reply to topic  [ 100 posts ]  Go to page Previous 13 4 5 6 7 Next

All times are UTC+03:00


Who is online

Users browsing this forum: No registered users and 1 guest


You cannot post new topics in this forum
You cannot reply to topics in this forum
You cannot edit your posts in this forum
You cannot delete your posts in this forum
You cannot post attachments in this forum

Search for:
Jump to:  
cron
Powered by phpBB® Forum Software © phpBB Limited