Все

Gemorroj · 2011.06.22 12:06

вот пока что есть.
там основная проблема в дублировании кода. нужно чистить.

Вложения

wapinet.zip

Gemorroj · 2011.06.24 16:22

еще полазил. удалил закрытие (закрываем просто кликнув в сторону), удалил автоскрытие, ИМХО не юзабельно. и так всякое.

Вложения

wapinet.zip

TLENS · 2011.06.24 16:25

Ну занимайся вообщем им.

Код:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63

Замени этот обьект

var blink = new function () { // Мигалка

this.works = 0; // Включен или нет

this.step = 0; // Шаг цвета

this.timeId = 0; // clearTimeout

this.start = function () {

notification = webkitNotifications.createHTMLNotification('notification.html'); // Создаем уведомление

notification.show(); // Показать

if (this.works) { // Если уже мигает то не надо запускать повторно

return;

}

this.works = true; // сохраняем Включен

//this.works = 1;

chrome.browserAction.setBadgeText({text: 'NEW'}); // Цепляем текст к иконке

blink.flip(); // Начнем мигить фоном текста

};

this.flip = function() { // Мигаем фоном текста путем последовательной замены его цвета

switch(this.step) {

case 0:

this.step++;

chrome.browserAction.setBadgeBackgroundColor({color: [255,255,255,255]});

break;

case 1:

this.step++;

chrome.browserAction.setBadgeBackgroundColor({color: [255,170,170,255]});

break;

case 2:

this.step++;

chrome.browserAction.setBadgeBackgroundColor({color: [255,85,85,255]});

break;

case 3:

this.step++;

chrome.browserAction.setBadgeBackgroundColor({color: [255,0,0,255]});

break;

case 4:

this.step++;

chrome.browserAction.setBadgeBackgroundColor({color: [255,85,85,255]});

break;

case 5:

this.step = 0;

chrome.browserAction.setBadgeBackgroundColor({color: [255,170,170,255]});

break;

default:

this.step = 1;

chrome.browserAction.setBadgeBackgroundColor({color: [255,255,255,255]});

break;

}

this.timeId = setTimeout(blink.flip, 100);

};

this.stop = function () { // Остановка

this.works = false;

this.step = 0;

clearTimeout(this.timeId);

chrome.browserAction.setBadgeText({text: ''}); // Чистим иконку

};

Gemorroj · 2011.06.24 17:02

опиши подробнее что изменилось?

TLENS · 2011.06.24 17:11

Изменил.
localStorage['popup']
на this.works
А то оно чет не правильно работает.
Меня та база уже накаляет с переведениям в стороки.
Пишу 1 - 0
А оно все равно не срабатывает.
Проблема была в том что при каждом новом сообщении включалась мигалка не зависимо от того мигает или нет.
И оно мигало не правильно тот цыкл ставит один цвет другой цыкл другой цвет.
Можно еще после 18 строки в обекте вставить для надежности clearTimeout(this.timeId);

Gemorroj · 2011.06.24 18:04

...
Добавлено спустя 1 минуту 24 секунды:
TLENS, просто там код дублирован в 100 мест. отсюда и проблемы. если бы ты вынес получение настроек в 1 место, то элементарно можно было бы все к булю привести, а так прям не знаешь с какой стороны подступиться.

Вложения

wapinet.zip

TLENS · 2011.06.24 18:09

Gemorroj написал:
прям не знаешь с какой стороны подступиться.

Что верно то верно.
Кстати никаких изменений.
Все равно пошел какой то сбой. Начал фон мигать как незнаю что. И не пойму в чем проблемма. Сейчас попробую вставить удаление таймаута
Добавлено спустя 1 минуту 35 секунд:
Видно по иконке что не один цыкл работает.
Мож вообще убрать ту мигалку и сделать правильно. Вставить туда число количество новых
Добавлено спустя 3 минуты :
Но не пойму. Такое началось после модов.
То тех пор все работало норм.
Добавлено спустя 4 минуты 1 секунду:
Сейчас консоль впишу затестирую

Gemorroj · 2011.06.24 18:16

ты скачивал последние модификации? показывает что загрузок 0 smile
проверь на новых версиях. потому как я не особо понимаю о чем ты говоришь)

TLENS · 2011.06.24 18:20

Все проблема решена при запуске попы цыкл не прекращает своих действий. (не срабатывает удаление таймаута)

Gemorroj · 2011.06.24 18:22

может svn заведем чтобы вместе код править?)

WAP Мастер Форум

#121 2011.06.22 12:06

Re: Переезд сервера

#122 2011.06.24 16:22

Re: Переезд сервера

#123 2011.06.24 16:25

Re: Переезд сервера

Код:

#124 2011.06.24 17:02

Re: Переезд сервера

#125 2011.06.24 17:11

Re: Переезд сервера

#126 2011.06.24 18:04

Re: Переезд сервера

#127 2011.06.24 18:09

Re: Переезд сервера

Gemorroj написал:

#128 2011.06.24 18:16

Re: Переезд сервера

#129 2011.06.24 18:20

Re: Переезд сервера

#130 2011.06.24 18:22

Re: Переезд сервера

Дополнительно