Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Paint rounded caps at the start and end of the Blockclock dial #309

Draft
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

johnny9
Copy link
Contributor

@johnny9 johnny9 commented Mar 31, 2023

Windows
Intel macOS
Apple Silicon macOS
ARM64 Android

@jarolrod
Copy link
Member

should base on 306

Copy link
Member

@jarolrod jarolrod left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Works really well, but gaps are painting too thick

master pr
Screen Shot 2023-04-10 at 2 29 50 PM Screen Shot 2023-04-10 at 2 27 30 PM

@GBKS
Copy link
Contributor

GBKS commented Apr 12, 2023

Rounded edges might work a little more nicely here.

image

I like how in the PR, the top hour indicator dot is covered. Looks cleaner, like the center line of segment starts at midnight/noon, and then the thickness of the line expands from that in all directions (hope that makes any sense, let me know if you want me to visualize it).

Also, is it possible to turn the hour indicator dots into actual circles? They seem like small segments of the block circle right now.

@hebasto hebasto added the UX Designers' opinions are required label Apr 17, 2023
@jarolrod jarolrod added Needs design review Designer's review needed Design suggestion Makes a suggestion on design labels Apr 17, 2023
@jarolrod
Copy link
Member

@GBKS we will have to patch the line caps within qt, we can do this for right now to get us a tiny bit closer to the designs.


if (-1 * nextAngle + 90 > m_animating_max_angle) {
nextAngle = -1 * m_animating_max_angle + 90;
// end the loop early
i = numberOfBlocks;
}

const qreal spanAngle = -1 * (startAngle - nextAngle) + gap;
path.arcTo(bounds, startAngle, spanAngle);
path.arcTo(gapBounds, nextAngle, gap * 2);
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

As a start, this makes the gaps close to what we want them to be

Suggested change
path.arcTo(gapBounds, nextAngle, gap * 2);
path.arcTo(gapBounds, nextAngle, gap);

@jarolrod
Copy link
Member

Ping @johnny9 any update on the sizing issue here?

Copy link
Member

@jarolrod jarolrod left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This still has an issue with the spacing for the divisions. Would be nice to iron that out as this is a visual improvement/refinement versus what's currently there:

Master PR
Screenshot 2023-09-08 at 1 25 12 AM Screenshot 2023-09-08 at 1 25 34 AM

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Design suggestion Makes a suggestion on design Needs design review Designer's review needed UX Designers' opinions are required
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants