Opened 11 years ago

Closed 11 years ago

Last modified 10 years ago

#816 closed (fixed)

icons for weather transports don't show

Reported by: aldafu Owned by: asterix
Priority: normal Milestone: 0.9
Component: roster Version:
Severity: normal Keywords: icon roster subscription
Cc: Blocked By:
Blocking: OS:

Description

I have weather.netlab.cz as transport in my roster. But cities added to the roster via this transport have a problem with subscription it seems, they show no icon at all and if i request authorization there is still no icon, but there is "(2)" appended to the name.

It seems these contacts don't need subscription, but gajim wont show the icon in the roster until they are subscribed. Oddly enough the right icon is displayed in the tooltip.

Change History (15)

comment:1 Changed 11 years ago by aldafu

  • Keywords subscription added; weather removed

Reading the source it's clear why the tooltip doesn't show the requested-icon. It just never bothers if the contact is subscribed or not.

Of course I could make the roster have the same behaviour, but then the requested-icon becomes useless. Maybe though there is another way to show the user that a contact isn't subscribed and still show the normal status icon of the contact.

comment:2 Changed 11 years ago by nk

"It seems these contacts don't need subscription, but gajim wont show the icon in the roster until they are subscribed."

Are you sure? then I can make requsted.png look like online.png?

where are you testing, so I can test..

about tooltip if what you say it is true, plz ticket against roster tooltip as that is a bug

comment:3 Changed 11 years ago by aldafu

The weather agent is on njs.netlab.cz. You then add a city as contact via this transport. In my case gmxx0007@….

You shouldn't make requested.png to online.png as the contact will change it's status according to the weather.

comment:4 Changed 11 years ago by nk

so what icon should I put for requested?

comment:5 Changed 11 years ago by aldafu

Don't put any other icon. What I'm looking for here is a way to not care about subscription and thus not use the requested-icon at all, but show the normal status icons instead. It seems Psi can do something like that (I havn't tried) as you cann see on the screenshots here: http://njs.netlab.cz/cz/help#08

comment:6 Changed 11 years ago by nk

is it fixed? if no, Yann?

comment:7 Changed 11 years ago by asterix

  • Milestone set to 0.9

should be fixed now with #710, #712 aldafu, could you confirm ?

comment:8 Changed 11 years ago by nk

  • Resolution set to fixed
  • Status changed from new to closed

reopen if not

comment:9 Changed 11 years ago by aldafu <acid.punk@…>

  • Resolution fixed deleted
  • Status changed from closed to reopened

Not fixed there is still no icon in the roster, but only in the tooltip.

comment:10 Changed 11 years ago by asterix

I'm not able to register to wether.netlab.cz with current svn

comment:11 Changed 11 years ago by aldafu <acid.punk@…>

You have to browse njs.netlab.cz with the disco browser and then register with the weather.netlab.cz service. After that you can add any city from http://weather.yahoo.com to your roster. In my case I added gmxx0007, which is Berlin, Germany.

comment:12 Changed 11 years ago by anonymous

  • Resolution set to invalid
  • Status changed from reopened to closed

ok I was able to register. and I se infos are subscription='from'and ask='suscribe'. I think it's bad and it's a transport pb. IMHO in such a case subscription should be 'to' and ask='none' To do that the transport should send a presence type=subscribed then a presence type=unsubscribe could you bug the transport's dev ?

comment:13 Changed 11 years ago by nk

  • Resolution invalid deleted
  • Status changed from closed to reopened

tolltip shows correct img. we should fix tooltip to show the sub icon

comment:14 Changed 11 years ago by nk

ok aldafu found new server which has newer version and does not suffer, so we should fix the tooltip and close

comment:15 Changed 11 years ago by anonymous

  • Resolution set to fixed
  • Status changed from reopened to closed
Note: See TracTickets for help on using tickets.