diff options
| author | Sebastian Geerken <devnull@localhost> | 2016-07-24 15:21:01 +0200 |
|---|---|---|
| committer | Sebastian Geerken <devnull@localhost> | 2016-07-24 15:21:01 +0200 |
| commit | f3a56bdc779b035b7adc37a2b61a3b3fbea9718d (patch) | |
| tree | 936d81dfecf21cac99cc3129806b5e4a130990a2 /dw/ooffloatsmgr.cc | |
| parent | 57fae805f2a9ee9f161dde38141f3682bd5eb847 (diff) | |
Fix update problem related to CSS 'clear'.
Diffstat (limited to 'dw/ooffloatsmgr.cc')
| -rw-r--r-- | dw/ooffloatsmgr.cc | 58 |
1 files changed, 46 insertions, 12 deletions
diff --git a/dw/ooffloatsmgr.cc b/dw/ooffloatsmgr.cc index 5e6570fa..5cc01b79 100644 --- a/dw/ooffloatsmgr.cc +++ b/dw/ooffloatsmgr.cc @@ -652,18 +652,8 @@ void OOFFloatsMgr::markSizeChange (int ref) vloat->dirty = true; DBG_OBJ_SET_BOOL_O (vloat->getWidget (), "<Float>.dirty", vloat->dirty); - assert (vloat->getWidget()->getWidgetReference() != NULL); - - int first = getOOFAwareWidget(vloat->generator)->index; - DBG_OBJ_MSGF ("resize.oofm", 1, "updating from %d", first); - - tbInfos->get(first)->getOOFAwareWidget() - ->updateReference (vloat->getWidget()->getWidgetReference() - ->parentRef); - - for (int i = first + 1; i < tbInfos->size(); i++) - tbInfos->get(i)->getOOFAwareWidget()->updateReference(0); - + updateGenerators (vloat); + floatRef = ref; DBG_OBJ_SET_NUM ("floatRef", floatRef); } @@ -672,6 +662,29 @@ void OOFFloatsMgr::markSizeChange (int ref) } /** + * \brief Update all generators which are affected by a given float. + */ +void OOFFloatsMgr::updateGenerators (Float *vloat) +{ + DBG_OBJ_ENTER ("resize.oofm", 0, "updateGenerators", "#%d [%p]", + vloat->index, vloat->getWidget ()); + + assert (vloat->getWidget()->getWidgetReference() != NULL); + + int first = getOOFAwareWidget(vloat->generator)->index; + DBG_OBJ_MSGF ("resize.oofm", 1, "updating from %d", first); + + tbInfos->get(first)->getOOFAwareWidget() + ->updateReference (vloat->getWidget()->getWidgetReference() + ->parentRef); + + for (int i = first + 1; i < tbInfos->size(); i++) + tbInfos->get(i)->getOOFAwareWidget()->updateReference(0); + + DBG_OBJ_LEAVE (); +} + +/** * `y` is given relative to the container. */ int OOFFloatsMgr::findTBInfo (int y) @@ -741,6 +754,8 @@ void OOFFloatsMgr::tellPosition1 (Widget *widget, int x, int y) getFloatsListsAndSide (vloat, &listSame, &listOpp, &side); ensureFloatSize (vloat); + int oldYReal = vloat->yReal; + // "yReal" may change due to collisions (see below). vloat->yReq = vloat->yReal = y; @@ -805,6 +820,25 @@ void OOFFloatsMgr::tellPosition1 (Widget *widget, int x, int y) DBG_OBJ_MSGF ("resize.oofm", 1, "vloat->yReq = %d, vloat->yReal = %d", vloat->yReq, vloat->yReal); + // In some cases, an explicit update is neccessary, as in this example: + // + // <body> + // <div id="a"> + // <div id="b" style="float:left">main</div> + // </div> + // <div id="c" style="clear:both">x</div> + // <div id="d">footer</div> + // </body> + // + // Without an explicit update, #c would keep an old value for extraSpace.top, + // based on the old value of vloat->yReal. + // + // Notice that #c would be updated otherwise, if it had at least one word + // content. + + if (vloat->yReal != oldYReal) + updateGenerators (vloat); + DBG_OBJ_LEAVE (); } |