https://bugs.kde.org/show_bug.cgi?id=463272

Bharadwaj Raju <bharadwaj.raju...@protonmail.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |bharadwaj.raju777@protonmai
                   |                            |l.com

--- Comment #9 from Bharadwaj Raju <bharadwaj.raju...@protonmail.com> ---
Correction (to comment 5 on https://bugs.kde.org/show_bug.cgi?id=468352#c5):

The tooltip doesn't in fact open at the same y-position as before. It opens
exactly 30 pixels upward/downward of it, in the "correct" direction (i.e. it
should actually be more than 30 pixels from it but in the same direction).

Here are some logs from adding some prints to dialog.cpp in PF:

-------------------
opening small tooltip
>>> syncToMainItemSize called
>>> setVisible: position is:  QPoint(0,840)
>>> syncToMainItemSize called
>>> repositionIfOffScreen: setting (x, y) =  0 ,  840
>>> updateVisibility: position is:  QPoint(0,840)
>>> syncToMainItemSize called
>>> repositionIfOffScreen: setting (x, y) =  0 ,  840
>>> updateVisibility: position is:  QPoint(0,840)
it opened with correct position
--------------------
now going to open tall tooltip
>>> syncToMainItemSize called
>>> setVisible: position is:  QPoint(0,810)
>>> syncToMainItemSize called
>>> repositionIfOffScreen: setting (x, y) =  0 ,  810
>>> updateVisibility: position is:  QPoint(0,810)
>>> repositionIfOffScreen: setting (x, y) =  0 ,  776
>>> repositionIfOffScreen: setting (x, y) =  0 ,  776
>>> syncToMainItemSize called
>>> repositionIfOffScreen: setting (x, y) =  0 ,  776
>>> updateVisibility: position is:  QPoint(0,776)
it opened with wrong position
---------------------
now reopening tall
>>> syncToMainItemSize called
>>> setVisible: position is:  QPoint(0,776)
>>> syncToMainItemSize called
>>> repositionIfOffScreen: setting (x, y) =  0 ,  776
>>> updateVisibility: position is:  QPoint(0,776)
>>> syncToMainItemSize called
>>> repositionIfOffScreen: setting (x, y) =  0 ,  776
>>> updateVisibility: position is:  QPoint(0,776)
correct position
---------------------
now opening small
>>> syncToMainItemSize called
>>> setVisible: position is:  QPoint(0,776)
>>> syncToMainItemSize called
>>> repositionIfOffScreen: setting (x, y) =  0 ,  806
>>> updateVisibility: position is:  QPoint(0,806)
>>> repositionIfOffScreen: setting (x, y) =  0 ,  840
>>> repositionIfOffScreen: setting (x, y) =  0 ,  840
>>> syncToMainItemSize called
>>> repositionIfOffScreen: setting (x, y) =  0 ,  840
>>> updateVisibility: position is:  QPoint(0,840)
wrong position
------------------

Turns out (from pixel-measuring screenshots) the wrong position for the tall
one is 810, exactly 30 pixels upwards from the correct position of the small
tooltip which is 840. Similarly, the wrong position for the small tooltip is
806, exactly 30 pixels downwards from the correct position of the tall tooltip
which is 776. I'm not sure what is making this 30 pixel thing happen.

Also note how it first says "updateVisibility: position is: <some wrong
position>" then soon after we get a similar line with the correct position, but
even after that has been printed the tooltip only actually shows up at the
first, wrong position. It's like the second time its position change isn't
having an effect...?

-- 
You are receiving this mail because:
You are watching all bug changes.

Reply via email to