Opened 11 years ago
Last modified 4 years ago
#9979 assigned enhancement
[ProcessController] kill dialog is not in the center of a screen
Reported by: | diver | Owned by: | nobody |
---|---|---|---|
Priority: | normal | Milestone: | R1.1 |
Component: | Kits/Interface Kit | Version: | R1/Development |
Keywords: | Cc: | ||
Blocked By: | Blocking: | ||
Platform: | All |
Description
if i remember correctly it used to be in the center before. Might be related to hrev45681.
Change History (4)
comment:1 by , 11 years ago
Type: | bug → enhancement |
---|
comment:2 by , 10 years ago
Owner: | changed from | to
---|---|
Status: | new → assigned |
comment:3 by , 4 years ago
Component: | Applications/ProcessController → Kits/Interface Kit |
---|
ProcessController can't really decide where the alert is shown, because its size is computed only in BAlert::Go() which also shows it. So, calling CenterOnScren results in the alert having its top-left corner on the center of the screen.
BAlert::_Prepare detects that the alert has a set position and does not change it. But there is no way to compute the correct position before _Prepare is called.
There is also no way to specify which window is the parent. It just uses the current thread and tests if that's a window (otherwise it does center itself on screen instead).
So, it looks like some API improvement to BAlert could be needed here.
comment:4 by , 4 years ago
Milestone: | R1 → R1.1 |
---|
The way it's positioned now is normal behavior for BAlert, it tries to position itself slightly offset from the window that spawned it, as best as allowed by proximity to the borders of the desktop. The above referenced commit doesn't change that, though it does change the size of the window slightly. In any case, PC didn't explicitly try to position itself either before or after said revision.