295424 - share code to set up tooltip content for unifinder, unifinderToDo

Status: RESOLVED FIXED

(Calendar :: Sunbird Only, defect)

Description

[gekacheka]

User-Agent:       Mozilla/5.0 (Windows; U; Windows NT 5.0; en-US; rv:1.7.8) Gecko/20050511 Firefox/1.0.4
Build Identifier: trunk

Cleanup:
  Rename changeTooltipForTask/Event to onMouseOverTask/Event
    to make clearer they are for event handlers.
  Move code from unifinderj.js and unifinderToDo.js to mouseoverPreviews.js
  Pass tooltip as parameter, avoid some document.getElementById calls.




Reproducible: Always

Steps to Reproduce:

Comment 1

[gekacheka]

Attachment: Clean up unifinder/unifinderToDo onMouseOver, share code

(patch -l -p 2 -i file.patch)

Clean up:
  Renamed ChangeToolTip... to onMouseOver... to make clearer 
    that they are event handlers when reading the js files.
  Pass tooltip so it doesn't have to be looked up with document.getEltById.
  Moved from unifinder.js and unifinderToDo.js to mouseoverPreviews.js
  Pulled out common code to setTooltipContent 
    (preparation to add more in other bugs).

testing:
  all three tooltips (grid, event list, task list) still work.

Comment 2

[gekacheka]

Patch depends on patch in bug 295418.

Comment 3

[gekacheka]

Comment on attachment 184462 [details] [diff] [review]
Clean up unifinder/unifinderToDo onMouseOver, share code

(split from bug 295146 as suggested.  thanks!)

Comment 4

[Michiel van Leeuwen (email: mvl+moz@)]

Comment on attachment 184462 [details] [diff] [review]
Clean up unifinder/unifinderToDo onMouseOver, share code

r=mvl

Comment 5

[Michiel van Leeuwen (email: mvl+moz@)]

patch checked in