Opened 7 years ago

Closed 7 years ago

#8753 closed bug (fixed)

[Debugger] sometimes opens with locked scrollbars

Reported by: diver Owned by: anevilyak
Priority: normal Milestone: R1
Component: Kits/Interface Kit Version: R1/Development
Keywords: Cc:
Blocked By: Blocking:
Has a Patch: no Platform: All

Description

This is hrev44361, gcc2hybrid.

After double clicking a thread in Teams window Debugger sometimes opens in a state where scrollbars are locked. In a different tabs (Images, Breakpoints) scrollbars are ok. Dragging splitters fixes it.

Attachments (1)

Debugger.png (28.3 KB) - added by diver 7 years ago.

Download all attachments as: .zip

Change History (14)

comment:1 Changed 7 years ago by bonefish

With "locked" you mean disabled, I assume. Which views are affected exactly? The threads list, the central source code view?

comment:2 Changed 7 years ago by diver

Yes, they are disabled. Affected views are: Threads tab, Variables tab and StackTraceView. Central source code view is empty so I can't check it.

comment:3 Changed 7 years ago by anevilyak

They visually appear disabled as well? I've occasionally seen similar behavior in other apps where a scrollbar appears "normal", but can't be clicked/dragged on either the arrows or the thumb. This is sometimes accompanied by some slight discoloration on parts of the scrollbar which leads me to suspect that what's going on is overlapping sibling views, but I haven't seen it in a consistent enough manner to analyze it. Also only seen in apps using layout so far for me at least.

comment:4 Changed 7 years ago by diver

No, scrollbar appears normal, but they don't react on mouse clicks. I can't see any discoloration, though. I can reproduce it easily in Debugger.

comment:5 Changed 7 years ago by diver

Resizing Debugger window a little bit also fixes them.

comment:6 Changed 7 years ago by diver

I can also move scrollbars using keyboard arrows but scrollbars redraw badly.

comment:7 Changed 7 years ago by bonefish

Component: Applications/DebuggerKits/Interface Kit
Owner: changed from bonefish to axeld

Changing component to Interface Kit. Sounds like a CLV or scroll bar/view issue. At least to my knowledge Debugger doesn't do anything special with respect to scrolling here.

comment:8 Changed 7 years ago by anevilyak

I wonder if this is a problem with UnarchiveSplitView... diver, can you try commenting out lines 378-388 in src/apps/debugger/user_interface/gui/team_window/TeamWindow.cpp and see if the behavior's still reproducible?

comment:9 Changed 7 years ago by diver

diff --git a/src/apps/debugger/user_interface/gui/team_window/TeamWindow.cpp b/src/apps/debugger/user_interface/gui/team_window/TeamWindow.cpp
index e436331..c0dc5f1 100644
--- a/src/apps/debugger/user_interface/gui/team_window/TeamWindow.cpp
+++ b/src/apps/debugger/user_interface/gui/team_window/TeamWindow.cpp
@@ -375,7 +375,7 @@ TeamWindow::LoadSettings(const GUITeamUISettings* settings)
        }

        BMessage archive;
-       if (teamWindowSettings.FindMessage("sourceSplit", &archive) == B_OK)
+/*     if (teamWindowSettings.FindMessage("sourceSplit", &archive) == B_OK)
                GUISettingsUtils::UnarchiveSplitView(archive, fSourceSplitView);

        if (teamWindowSettings.FindMessage("functionSplit", &archive) == B_OK)
@@ -386,7 +386,7 @@ TeamWindow::LoadSettings(const GUITeamUISettings* settings)

        if (teamWindowSettings.FindMessage("threadSplit", &archive) == B_OK)
                GUISettingsUtils::UnarchiveSplitView(archive, fThreadSplitView);
-
+*/
        if (teamWindowSettings.FindMessage("imageListView", &archive) == B_OK)
                fImageListView->LoadSettings(archive);

I still can reproduce it with this change.

Changed 7 years ago by diver

Attachment: Debugger.png added

comment:10 Changed 7 years ago by diver

It appears that scrollbar actually reacts on clicks, but I have to click right under the scrollbar, though. Clicking on red area - presses arrow buttons. Clicking on green part - moves knob to the other side.

comment:11 Changed 7 years ago by diver

comment:12 Changed 7 years ago by anevilyak

Owner: changed from axeld to anevilyak
Status: newin-progress

comment:13 Changed 7 years ago by anevilyak

Resolution: fixed
Status: in-progressclosed

Fixed in hrev44366.

Note: See TracTickets for help on using tickets.