View Issue Details
| ID | Project | Category | View Status | Date Submitted | Last Update |
|---|---|---|---|---|---|
| 0017003 | Scribus | Internal | public | 2023-08-14 19:05 | 2023-09-04 19:26 |
| Reporter | ale | Assigned To | |||
| Priority | normal | Severity | minor | Reproducibility | N/A |
| Status | new | Resolution | open | ||
| Product Version | 1.7.0.svn | ||||
| Summary | 0017003: when creating a shape, void ColorPalette::setCurrentItem(PageItem* item) is called twice | ||||
| Description | i've added "qDebug() << "setup" << item->itemName();" inside of the function and it gets called twice with the same itemName() when creating a shape. then, it gets called twice when i change the "fill mode " in the "Fill" tab. | ||||
| Tags | No tags attached. | ||||
| Patch | No | ||||
|
|
It doesn't affect only the color palette, it happens for all palettes that set a PageItem. Almost all palettes using two functions to set a PageItem "setCurrentItem(PageItem*)" and "handleSelectionChanged()". I could figure out that the ScribusDoc::changed() is called twice. Because of that, the connected slots set the PageItem via handleSelectionChanged() again. connect(m_doc->m_Selection, SIGNAL(selectionChanged()), this, SLOT(handleSelectionChanged())); connect(m_doc, SIGNAL(docChanged()), this, SLOT(handleSelectionChanged())); At the moment I could not figure out why docChanged() is emitted twice. |
