Skip to content

Fix button offset when window resizes#354

Merged
LmeSzinc merged 8 commits intoLmeSzinc:devfrom
JerryLingjieMei:button_offset
Mar 20, 2024
Merged

Fix button offset when window resizes#354
LmeSzinc merged 8 commits intoLmeSzinc:devfrom
JerryLingjieMei:button_offset

Conversation

@JerryLingjieMei
Copy link
Contributor

#345 with button offset determined by whether the rhombus technique point 1 is filled or not.

@JerryLingjieMei JerryLingjieMei changed the title Button offset Fix button offset when window resizes Mar 4, 2024
Comment on lines +37 to +44
TECHNIQUE_POINT_LOOSE_0 = build_technique_points_loose(TECHNIQUE_POINT_0)
TECHNIQUE_POINT_LOOSE_1 = build_technique_points_loose(TECHNIQUE_POINT_1)

TECHNIQUE_POINT_STRICT_1 = build_technique_points_strict(TECHNIQUE_POINT_1)
TECHNIQUE_POINT_STRICT_2 = build_technique_points_strict(TECHNIQUE_POINT_2)
TECHNIQUE_POINT_STRICT_3 = build_technique_points_strict(TECHNIQUE_POINT_3)
TECHNIQUE_POINT_STRICT_4 = build_technique_points_strict(TECHNIQUE_POINT_4)
TECHNIQUE_POINT_STRICT_5 = build_technique_points_strict(TECHNIQUE_POINT_5)
Copy link
Owner

Choose a reason for hiding this comment

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

Add TECHINQUE_POINT_1.SEARCH.png to enlarge search area, dont dynamically build buttons

Args:
button(Button, tuple): Button instance or area tuple.
copy:
offset:
Copy link
Owner

Choose a reason for hiding this comment

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

Remove unused argments

@LmeSzinc LmeSzinc added bug / 缺陷 Something isn't working good first issue / 首次贡献 Good for newcomers labels Mar 5, 2024
@JerryLingjieMei
Copy link
Contributor Author

Ready to merge, also fixed technique_pt_5 figure.

Copy link
Owner

@LmeSzinc LmeSzinc left a comment

Choose a reason for hiding this comment

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

Remove unused assets, keep only TECHNIQUE_POINT_[012345], only TECHNIQUE_POINT_[01] have SEARCH attribute while the rest don't

color (tuple): RGB.
threshold: 255 means colors are the same, the lower the worse.
count (int): Pixels count.
offset: if the button is offset
Copy link
Owner

Choose a reason for hiding this comment

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

Remove unused argments

@LmeSzinc LmeSzinc changed the base branch from master to dev March 20, 2024 09:54
@LmeSzinc LmeSzinc merged commit d9fb903 into LmeSzinc:dev Mar 20, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug / 缺陷 Something isn't working good first issue / 首次贡献 Good for newcomers

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants