Ticket #1179 (closed bug: fixed)

Opened 6 years ago

Last modified 5 years ago

event pane layers with padding drag funny

Reported by: tschaub Owned by: euzuro
Priority: critical Milestone: 2.7 Release
Component: Layer.EventPane Version: 2.5
Keywords: Cc:
State: Complete

Description

Attachments

killMarginPadding.patch Download (324 bytes) - added by euzuro 5 years ago.
once we have schaub's classname patch in, we need to automatically assign the main map div a 'olMap' classname, and then this mod to the style.css will fix the stated problem.
killMarginPadding.2.patch Download (0.8 KB) - added by euzuro 5 years ago.
ok. classname patch is in, so here is the big poobah.
noPaddingMargin.patch Download (1.6 KB) - added by crschmidt 5 years ago.
same patch, with a test for olMap class

Change History

Changed 5 years ago by euzuro

  • milestone set to 2.7 Release

this sounds like #1116 to me. can someone corroborate that?

my brain is saying "close all these tickets in favor of one big ticket that explains the 'dont use x,y,z styles on your map div or any parent divs thereof' issue... which hopefully can either be solved somehow or (more likely) we can at least print a clear notification somewhere in the code/documentation so people know about this"

Changed 5 years ago by crschmidt

  • priority changed from minor to critical

Turn off padding in CSS. Force it to 0px because we own it.

Changed 5 years ago by euzuro

once we have schaub's classname patch in, we need to automatically assign the main map div a 'olMap' classname, and then this mod to the style.css will fix the stated problem.

Changed 5 years ago by euzuro

  • state set to Needs More Work

Changed 5 years ago by euzuro

ok. classname patch is in, so here is the big poobah.

Changed 5 years ago by euzuro

  • state changed from Needs More Work to Review

all tests pass ff2, ie7* (despite your mockery).

please review

*excepting overviewmap which has speed issue

Changed 5 years ago by euzuro

when this is approved and committed, I can go ahead and make patch for #1635

Changed 5 years ago by euzuro

  • status changed from new to assigned

Changed 5 years ago by crschmidt

same patch, with a test for olMap class

Changed 5 years ago by crschmidt

  • state changed from Review to Commit

Erik,

Checked it out, made sure that the Map div now gets an olMap class (test included in my most recent patch) and that the CSS rules are applied on examples/lite.html. Tested it by setting margin/padding of 1000px on the map element with no problems. 'hit it.'

Changed 5 years ago by euzuro

  • status changed from assigned to closed
  • state changed from Commit to Complete
  • resolution set to fixed

(In [7584]) patch to make sure main openlayers map div has neither padding nor margin. done via css thanks to tim's fabulous new class manip patch. r=cr5 (Closes #1179)

Changed 5 years ago by euzuro

accidentally added a change to Events.js with r7584 -- reverted in r7585

Note: See TracTickets for help on using tickets.