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

Feature/smoother collapsing tiles #739

Merged

Conversation

ishaanthenerd
Copy link
Contributor

Overview

Resolves "Bug: Make Collapse/Expanding of Transfer Bank and Semester Tiles Smoother · Issue #433 · UTDNebula/planner"

What Changed

I added code to smoothly transition the collapsing and expanding of the semester tiles and the transfer bank. Although they now have fixed sizes when closed, they match the original sizes almost exactly.

Other Notes

I noticed that venv/bin/activate was acting strangely... don't merge changes related to that, only from SemesterTile and TransferBank.

@ishaanthenerd ishaanthenerd requested a review from a team as a code owner November 29, 2023 03:59
smoother for the semester and transfer bank tiles
@Jake3231 Jake3231 force-pushed the feature/smoother-collapsing-tiles branch from 5311049 to f557573 Compare February 6, 2024 21:23
@Jake3231
Copy link
Contributor

Jake3231 commented Feb 6, 2024

@ishaanthenerd Looks like the issues with the unexpected changes are that the IDE included a venv folder in the project. I also rebased to the latest develop to catch up with those changes. Should be ready to test.

@Jake3231 Jake3231 requested a review from jasonappah February 6, 2024 21:30
Copy link
Contributor

@Jake3231 Jake3231 left a comment

Choose a reason for hiding this comment

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

@ishaanthenerd Please run prettier and resolve other linting issues.

Copy link
Member

@cubetastic33 cubetastic33 left a comment

Choose a reason for hiding this comment

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

@ishaanthenerd Please run prettier and eslint and re-request a review!

@jasonappah jasonappah merged commit 828c489 into UTDNebula:develop Feb 13, 2024
3 of 6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Bug: Make Collapse/Expanding of Transfer Bank and Semester Tiles Smoother
5 participants