kindred/create
1
1
Fork 0
Code Issues 53 Pull Requests Actions Packages Projects 9 Releases 2 Wiki Activity

[Bugfix]Spreadsheet: fix multi-screen management of QtColorPicker

Browse Source 
+ set parent when instantiating
This commit is contained in:
0penBrain
2022-10-13 17:47:40 +02:00
committed by Chris Hennes
parent 5fcc1c3d03
commit 0a12d435a2
2 changed files with 6 additions and 5 deletions
Download Patch File Download Diff File Expand all files Collapse all files

4
src/Mod/Spreadsheet/Gui/Workbench.cpp
View File

@@ -82,7 +82,7 @@ void Workbench::activated()
if (!fgList.empty())
foregroundColor = fgList[0];
else {
foregroundColor = new QtColorPicker();
foregroundColor = new QtColorPicker(bar);
foregroundColor->setObjectName(QString::fromLatin1("Spreadsheet_ForegroundColor"));
foregroundColor->setStandardColors();
foregroundColor->setCurrentColor(palette.color(QPalette::WindowText));
@@ -97,7 +97,7 @@ void Workbench::activated()
if (!bgList.empty())
backgroundColor = bgList[0];
else {
backgroundColor = new QtColorPicker();
backgroundColor = new QtColorPicker(bar);
backgroundColor->setObjectName(QString::fromLatin1("Spreadsheet_BackgroundColor"));
backgroundColor->setStandardColors();
backgroundColor->setCurrentColor(palette.color(QPalette::Base));

7
src/Mod/Spreadsheet/Gui/qtcolorpicker.cpp
View File

@@ -3,6 +3,7 @@
** This file is part of a Qt Solutions component.
**
** Copyright (c) 2009 Nokia Corporation and/or its subsidiary(-ies).
** Modified 2022 by 0penBrain under LGPL : fix issues about popup positioning
**
** Contact: Qt Software Information (qt-info@nokia.com)
**
@@ -314,7 +315,7 @@ void QtColorPicker::buttonPressed(bool toggled)
if (!toggled)
return;
const QRect desktop = QApplication::primaryScreen()->geometry();
const QRect desktop = QApplication::activeWindow()->geometry();
// Make sure the popup is inside the desktop.
QPoint pos = mapToGlobal(rect().bottomLeft());
@@ -323,8 +324,8 @@ void QtColorPicker::buttonPressed(bool toggled)
if (pos.y() < desktop.top())
pos.setY(desktop.top());
if ((pos.x() + popup->sizeHint().width()) > desktop.width())
pos.setX(desktop.width() - popup->sizeHint().width());
if ((pos.x() + popup->sizeHint().width()) > desktop.right())
pos.setX(desktop.right() - popup->sizeHint().width());
if ((pos.y() + popup->sizeHint().height()) > desktop.bottom())
pos.setY(desktop.bottom() - popup->sizeHint().height());
popup->move(pos);