Last Comment Bug 679908 - probing.png is missing
: probing.png is missing
Product: Thunderbird
Classification: Client Software
Component: Theme (show other bugs)
: unspecified
: All All
-- normal (vote)
: Thunderbird 23.0
Assigned To: Richard Marti (:Paenglab)
Depends on:
  Show dependency treegraph
Reported: 2011-08-17 15:40 PDT by Philipp Kewisch [:Fallen]
Modified: 2013-05-13 06:41 PDT (History)
4 users (show)
See Also:
Crash Signature:
QA Whiteboard:
Iteration: ---
Points: ---

patch (2.98 KB, patch)
2013-04-23 10:55 PDT, Richard Marti (:Paenglab)
bwinton: review+
Details | Diff | Splinter Review

Description User image Philipp Kewisch [:Fallen] 2011-08-17 15:40:33 PDT
The new account wizard references probing.png:^[^\0]*%24&hitlimit=&tree=comm-central

which is nowhere to be found in the tree:
Comment 1 User image Richard Marti (:Paenglab) 2013-01-03 11:28:11 PST
It looks probing.png was never in the tree. Should we use loading.png instead (
Comment 2 User image Blake Winton (:bwinton) (:☕️) 2013-02-03 11:22:51 PST
Given that the others are all coloured circles, I think we want a new image for this.

Andreas, could you whip some icons up that are similar to the ones at|insecure|failed%29.png&tree=comm-central&hint= ?

Comment 3 User image Richard Marti (:Paenglab) 2013-04-21 04:00:02 PDT
It looks like Bug 549045 has removed the icons from XUL file. I think we can remove also the rules from CSS file.

Blake, what do you think about this?
Comment 4 User image Blake Winton (:bwinton) (:☕️) 2013-04-22 10:49:43 PDT
It did?  Are you _sure_?  Cause that doesn't sound right…
Comment 5 User image Richard Marti (:Paenglab) 2013-04-22 11:39:38 PDT
When you look in then on incomingarea the class="icon" is here removed: and in actual file where is no icon class:

The same is for outgoingarea.

Also when you look at Ben's screenshot where is no icon to show the status:

The only occurence of "icon" is here: and line 1433. But this creates the class warningicon.

For this I would say this icons are no more used.
Comment 6 User image Blake Winton (:bwinton) (:☕️) 2013-04-22 13:46:13 PDT
Okay, that sounds good, then.  (Unless we want to re-add them…  ;)
Comment 7 User image Richard Marti (:Paenglab) 2013-04-23 10:55:25 PDT
Created attachment 740930 [details] [diff] [review]

I removed the rules for the class icon. On osx where was a failed.png file but it wasn't referenced in and linux and windows didn't have this file.

Blake, please can you recheck if this class is really not used? I checked probing and weak in MXR and haven't found any appearance where the icon class becomes this state. The same with strong.
Comment 8 User image Blake Winton (:bwinton) (:☕️) 2013-05-12 19:22:11 PDT
Comment on attachment 740930 [details] [diff] [review]

Yeah, I agree, the classes are really not used anywhere.  r=me.

Comment 9 User image Ryan VanderMeulen [:RyanVM] 2013-05-13 06:40:01 PDT

Note You need to log in before you can comment on or make changes to this bug.