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.

Sign up for GitHub

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

Jump to bottom

Resizing test #2327

Open
wants to merge 4 commits into
base: develop
Choose a base branch
from
Open

Resizing test #2327

wants to merge 4 commits into from

Conversation

namnguyen20999
Copy link
Member

@namnguyen20999 namnguyen20999 commented Dec 12, 2024 •
edited
Loading

What type of PR is this? (check all applicable)

  • Refactor
  • Feature
  • Bug Fix
  • Optimization
  • Documentation Update

Description

Added logic to track changes in the element’s bounding box dimensions to verify if the element has grown or shrunk after resizing operations.

Checklist

We encourage you to keep the code coverage percentage at 80% and above.

  • Does this solution meet the acceptance criteria of the related issue?
  • Is the related issue checklist completed?
  • Does this PR adds unit tests for the developed code? If not, why?
  • End-to-End tests have been added or updated?
  • Was the documentation updated, or a dedicated issue for documentation created? (If applicable)
  • Is the release notes updated? (If applicable)

@namnguyen20999 namnguyen20999 changed the title demo for resizing test Resizing test Dec 12, 2024
@github-actions GitHub Actions
Copy link
Contributor

github-actions bot commented Dec 12, 2024 •
edited
Loading

☂️ Python Coverage

current status: ✅

Overall Coverage

Lines Covered Coverage Threshold Status
19485 16952 87% 0% 🟢

New Files

No new covered files...

Modified Files

No covered modified files...

updated for commit: 13d8506 by action🐍

@namnguyen20999 namnguyen20999 self-assigned this Dec 12, 2024
@namnguyen20999 namnguyen20999 added the 🧪Testing Related to tests and other QA activities. label Dec 12, 2024
@namnguyen20999 namnguyen20999 marked this pull request as ready for review December 12, 2024 17:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@dinhlongviolin1 dinhlongviolin1 Awaiting requested review from dinhlongviolin1

@FabienLelaquais FabienLelaquais Awaiting requested review from FabienLelaquais

@FredLL-Avaiga FredLL-Avaiga Awaiting requested review from FredLL-Avaiga

At least 1 approving review is required to merge this pull request.

Assignees

@namnguyen20999 namnguyen20999

Labels
GUI: Front-End 🧪Testing Related to tests and other QA activities.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@namnguyen20999