Opened 18 years ago

Closed 18 years ago

Last modified 18 years ago

#574 closed bug (fixed)

[Deskbar] edges are not drawn correctly when docked at the bottom

Reported by: j_freeman@… Owned by: jackburton
Priority: low Milestone: R1
Component: - General Version:
Keywords: Cc: diver
Blocked By: Blocking:
Platform: All

Description

Dock the Deskbar at the bottom of your screen, and you will see the top edge, where team buttons are not present, is not correctly drawn in 3D as it is in R5.

Tested in revision 17370 (and many, many previous ones) under VMware and real hardware, in various video modes and colorspaces.

Attachments (1)

deskbar.png​ (7.4 KB ) - added by j_freeman@… 18 years ago.
Illustration

Download all attachments as: .zip

Change History (14)

comment:1 by j_freeman@…, 18 years ago

severity: normal → trivial

comment:2 by j_freeman@…, 18 years ago

Summary: [Deskbar] 3D edges are not drawn correctly when docked at the bottom → [Deskbar] edge is not drawn correctly when docked at the bottom

by j_freeman@…, 18 years ago

Attachment: deskbar.png​ added

Illustration

comment:3 by j_freeman@…, 18 years ago

Summary: [Deskbar] edge is not drawn correctly when docked at the bottom → [Deskbar] edges are not drawn correctly when docked at the bottom

comment:4 by j_freeman@…, 18 years ago

Another note: The bottom edge isn't even drawn; at least the top edge is drawn (just with the wrong color). This is also shown in the illustration attached.

comment:5 by diver, 18 years ago

Cc: diver added

comment:6 by jackburton, 18 years ago

Owner: changed from axeld to jackburton

comment:7 by jackburton, 18 years ago

I think I found out what's wrong: BMenuBar uses BMenu::Draw() to draw the menubar, and our version of it doesn't draw the border around the menu frame, because we let the menuwindow handle it. We should probably change (back) the menuwindow's look to B_NO_BORDER_WINDOW_LOOK and change the menu drawing code to draw the border itself. I can do that.

comment:8 by jackburton, 18 years ago

Component: Servers → Kits

comment:9 by axeld, 18 years ago

Yes, please, that sounds like the best solution :-)

comment:10 by jackburton, 18 years ago

Status: new → assigned

comment:11 by jackburton, 18 years ago

Fixed in hrev18459

comment:12 by jackburton, 18 years ago

Resolution: → fixed

comment:13 by jackburton, 18 years ago

Status: assigned → closed
Note: See TracTickets for help on using tickets.