Opened 2 years ago

Last modified 2 years ago

#13174 assigned enhancement

[CodyCam] fix incorrect use of Layout API (easy)

Reported by: diver Owned by: humdinger
Priority: normal Milestone: Unscheduled
Component: Applications/CodyCam Version: R1/Development
Keywords: Cc:
Blocked By: Blocking:
Has a Patch: no Platform: All

Attachments (1)

codycam.png (55.3 KB) - added by diver 2 years ago.

Download all attachments as: .zip

Change History (16)

comment:1 Changed 2 years ago by CodeforEvolution

I looked in CodyCam's code and it seems to already be using the layout API, or did I accidentally miss a section of code?

Last edited 2 years ago by CodeforEvolution (previous) (diff)

comment:2 Changed 2 years ago by diver

Hmm, this is odd. I can't reproduce GUI problems (overlapping) in vmware with 24pt. font but it looks messed up on real hw with HiDPI. The same with #13164 and #13166. The only difference is resolution.

comment:3 Changed 2 years ago by CodeforEvolution

In that case, I'd assume it must be a general problem with sizing when using HiDPI displays. In that case, support for HiDPI is going to have to be implemented in some form eventually...

comment:4 Changed 2 years ago by diver

Description: modified (diff)

Changed 2 years ago by diver

Attachment: codycam.png added

comment:5 Changed 2 years ago by diver

Owner: changed from mmu_man to humdinger
Status: newassigned

comment:6 Changed 2 years ago by diver

Description: modified (diff)

comment:7 Changed 2 years ago by axeld

CodyCam is using the layout API, but it is using it incorrectly; the BBox comes with its own layout that makes sure the label is taken into account. However, the code sets a new layout on the BBox, causing the issue -- it uses a hard coded top offset that hides this bug in the 'normal' font size case.

comment:8 Changed 2 years ago by pulkomandy

And the fix, if anyone wants to tackle it, is to add an extra BView as the BBox only children, add the grid layout to that, and add the children there.

comment:9 Changed 2 years ago by pulkomandy

Summary: [CodyCam] rewrite to use Layout API[CodyCam] fix incorrect use of Layout API (easy)

comment:10 Changed 2 years ago by diver

Does hrev50853 also fix Video source view alignment?

comment:11 Changed 2 years ago by humdinger

Not sure what exactly you mean. The top box that's supposed to show the video source is centered horizontally in the window. It keeps centered when resizing the window.

comment:12 Changed 2 years ago by diver

Shouldn't it spread horizontally across the window or its geometry supposed to be like that?

comment:13 Changed 2 years ago by humdinger

I guess we'll only know if we get support for a real video source... if it can stretch, we can re-enable vertical resizing and scale the video box. Until then it's fixed size and aspect.

comment:14 Changed 2 years ago by pulkomandy

The video view should rescale when it manages to find a video source. You need one of the working "sonix" webcams, or I can send you a patched UVC driver, which somewhat works (shows only a black area for me, but at least it detects the webcam) for testing.

comment:15 Changed 2 years ago by humdinger

Sure PulkoMandy, I can give it a try. Maybe upload it somewhere.

Note: See TracTickets for help on using tickets.