Ticket #8576 (new defect)

Opened 14 months ago

Last modified 14 months ago

Mixing percentage with other width units causes the layout to glitch when resizing columns

Reported by: jaredj Owned by: BryanForbes
Priority: highest Milestone: future
Component: DojoX Grid Version: 1.2.3
Severity: normal Keywords:
Cc:

Description

Grid errors and scrambles view in tests/test_edit_dijit.html when adjusting column width.

I've tried this in FF2 and IE 7. In FF2 (And safari 3.1), the view becomes scrambled after a resize. See attached picture 1.

On IE 7 just clicking on the column to resize it throws JS errors:

Runtime error: Invalid argument

In either case, that testcase fails terribly.

Attachments

Resize_priority_mangles_grid.jpg (159.7 kB) - added by jaredj 14 months ago.
Screenshots showing after resize of field (in this case priority), grid becomes scrambled.

Change History

Changed 14 months ago by jaredj

Screenshots showing after resize of field (in this case priority), grid becomes scrambled.

Changed 14 months ago by BryanForbes

  • summary changed from Grid errors and scrambles view in tests/test_edit_dijit.html when adjusting column width. to Mixing percentage with other width units causes the layout to glitch when resizing columns
  • milestone changed from tbd to future

This is because the test mixes % widths with em widths and that is failing right now. Because we're so close to 1.3, I don't want to go in and muck around with the resizing code and potentially break it. I'm checking in a fix to change the width of the message to 10 instead of 100%. I'm also going to move this to "future".

Changed 14 months ago by BryanForbes

(In [16550]) * Changed setValue and setAttribute to use .attr(). !strict

  • Changed from using time since the epoch to Date objects in the test data.
  • Updated test_edit_dijit.html to use 10ems instead of 100% for the message width (refs #8576).
Note: See TracTickets for help on using tickets.