Ticket #3455 (closed defect: fixed)

Opened 14 months ago

Last modified 12 months ago

dojo.coords().y (and probably x), wrong value on Safari3 when page scrolled

Reported by: bill Owned by: sjmiles
Priority: normal Milestone: 0.9
Component: HTML Version: 0.9
Severity: normal Keywords: 4dijit
Cc:

Description (last modified by bill) (diff)

dojo.coords(node, true) is supposed to return the absolute position of a node.

It works correctly on FF mac, but on Safari3, the value returned depends on how much you scroll the page (as though I had called dojo.coords(node, false), I guess)

Tested on test_Dialog.html, for the lower link to show a TooltipDialog?. The TooltipDialog? on Safari shows up in the wrong place when the page is scrolled.

Change History

Changed 14 months ago by bill

(In [9189]) Fix/workaround drop down positioning on Safari on scrolled page (refs #3455)

Changed 14 months ago by bill

(In [9193]) Ah, dojo.coords() (x,y) is the absolute position, and (t,l) is position relative to the nearest position:relative/absolute ancestor.

Fixes #3465, and removes workaround for #3455 (refs #3455), so that bug has to be fixed for real.

Changed 14 months ago by bill

  • keywords 4dijit added
  • description modified (diff)
  • milestone changed from 0.9 to 0.9beta

Changed 14 months ago by bill

  • description modified (diff)

Changed 14 months ago by bill

(In [9350]) Specify icons in buttons via CSS rather than <img> tag. The CSS for icons is either in tundra.css (for the editor icons), or in the customer's css file (for custom icons).

Toolbar buttons have an icon and text defined but you can control which shows up via CSS rules (the text needs to show up if dijit_a11y is set). Haven't created those rules yet though, so for now both icon and text show up.

Need to doublecheck a11y stuff; I think I might have broken it.

Refs #3455.

When MenuItem? is converted to extend Button it will inherit this method of setting icons.

Changed 14 months ago by bill

(In [9354]) Don't need these individual files anymore; using editor.gif sprite instead. Refs #3455.

Changed 13 months ago by peller

  • owner changed from BryanForbes to sjmiles

Changed 13 months ago by sjmiles

  • status changed from new to closed
  • resolution set to worksforme

Can't reproduce, please provide a test case if you still have a problem.

Changed 13 months ago by bill

  • status changed from closed to reopened
  • resolution deleted

Not sure what you mean; I listed a testcase in the original bug description. Would you like a different testcase?

Changed 13 months ago by sjmiles

We messed up, sorry. Can reproduce it now.

Changed 12 months ago by sjmiles

  • status changed from reopened to closed
  • resolution set to fixed

(In [10186]) Make toPixelValue translate 'medium' to 4, hack refs #3874.
Repair scrolling adjustment in _abs, fixes #3455.

Note: See TracTickets for help on using tickets.