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

Animated collapse #672

Merged
merged 36 commits into from
Apr 21, 2023
Merged

Animated collapse #672

merged 36 commits into from
Apr 21, 2023

Conversation

skyjune0824
Copy link
Contributor

Issue

622

Photos
cl1
cl2

Additional Info

Collapse button no animated!

Also, the sidebar is animated.

@skyjune0824 skyjune0824 added the front-end front-end related (usually in vue.js) label Mar 24, 2023
@jonha1
Copy link
Contributor

jonha1 commented Mar 31, 2023

Love the feature and how useful it is but when it is in dark mode, the arrows to pull out that tab are difficult to see. You can make it a different color maybe blue. It feels like there is a slight delay when closing and opening the tab unless that is intended.

@skyjune0824
Copy link
Contributor Author

Fixed the color, followed by the scheme. It is clearly visible when dark mode is on now.
Also, for the delay when opening, it is mainly because loading time of course selection is relatively long. I can't do anything with it right now, so might consider to change the data structure later.

zhangn7
zhangn7 previously approved these changes Mar 31, 2023
Copy link

@zhangn7 zhangn7 left a comment

Choose a reason for hiding this comment

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

I tested what reviewers said previously about the bugs and I can add courses while opening the tab, and the animated collapse button also looks great in dark mode. It is noticeable in light and dark modes.

@skyjune0824 skyjune0824 requested a review from mlammon11 March 31, 2023 20:50
Copy link
Contributor

@mlammon11 mlammon11 left a comment

Choose a reason for hiding this comment

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

Changes from before were mended. However, I see that the color of the course list on light mode is blue, which is a bit hard on the eyes, maybe change it back to black like it currently is. Additionally, I believe that the default action of clicking on the schedule tab is to have the course search on the left instead of automatically closing it.

font-size and color was re-defined, so I fixed it.
Defalt status of sidebar is now 'open'
@skyjune0824 skyjune0824 requested review from mlammon11 April 4, 2023 20:46
charleschae12
charleschae12 previously approved these changes Apr 4, 2023
Copy link
Contributor

@charleschae12 charleschae12 left a comment

Choose a reason for hiding this comment

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

Looks GOOD.

Animatino is now rotating for only Y-axis
@skyjune0824
Copy link
Contributor Author

Finalized animation, and color scheme.
It looks fine now :)

캡처

mlammon11
mlammon11 previously approved these changes Apr 7, 2023
Copy link
Contributor

@mlammon11 mlammon11 left a comment

Choose a reason for hiding this comment

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

Changes were mended, everything looks great and is functional to the user. Good job

chavab0809
chavab0809 previously approved these changes Apr 7, 2023
Copy link

@chavab0809 chavab0809 left a comment

Choose a reason for hiding this comment

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

Just looked through all the change requests and verified that they all work now. Everything looks good!

* Creating Contribution Guide (#644)

This merge is a test to see if Lighthouse CI is being honest to us or not

Co-authored-by: dorian451 <112524240+dorian451@users.noreply.github.com>

* Fix docker (#655)

* Update requirements.txt

* Update requirements.txt

* Color Mode Button/Notification

-Changed the notification button to a switch
-Added brief text in front so user knows function
- Moved toast notification to top-center of screen

* Delete workspace.xml

* Dropdown

added drop down to condense all 3 functions of button/moon

* Drop Down Funcitonal

Drop down logic is now functional, still need to add more styling

* Updated Notifications and functions

Changed the text and color of notifications. added some logic for functions. currently have one bug where sometimes the first mode clicked is the opposite.

* Navbar highlight current page (#632)

* add highlight to currently selected page in navbar
(this took a lot longer than it should have)

* make border thicker

* fix navbar only being highlighted on top-level pages

* beep boop Github Action Master Branch CI Prettier hard at work

* Font and color branch (#625)

* Update CourseExplorer.vue

Changed the color of the title same ad other pages.

* Font_and_color_changed

* beep boop Github Action Master Branch CI Prettier hard at work

* See if  I can't specify sqlAlchemy

This is a bad news. Since the localhost Docker appears to require us to specify the version.

* remove 2020 data

* Updated Toggle Style Logic

Believe to have resolved bug, now working as intended. Need to change text color on dark mode so that its more readable

* More updates

comments to understand logic and other updates

* Cleaning up

just cleaning up code and testings

* New logic update

updated function logic to hopefully fix errors

* Fixed Dropdown Colors

Dark mode hover color changed so that the text is more visible to the user

* update to running fall 2023 instead pf spring 2023

* change to get summer class info

* fixed a summer address link

* Adding Current Item Highlight

trying to change background color based on currently selected item

* changes explore icon (#661)

changes explore icon

* beep boop Github Action Master Branch CI Prettier hard at work

* Update CourseListing.vue (#667)

* beep boop Github Action Master Branch CI Prettier hard at work

* Current Mode Selected Feature Update

Updated some logic and testing, not fully complete

* More updates

Still working out logic, changed color to what will most likely be close to final color

* Clean up

removing unnecessary code

* Bump json5 from 2.1.3 to 2.2.3 in /tests/integration (#638)

Bumps [json5](https://github.com/json5/json5) from 2.1.3 to 2.2.3.
- [Release notes](https://github.com/json5/json5/releases)
- [Changelog](https://github.com/json5/json5/blob/main/CHANGELOG.md)
- [Commits](json5/json5@v2.1.3...v2.2.3)

---
updated-dependencies:
- dependency-name: json5
  dependency-type: indirect
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* Working on fixing cookies

reworking logic to fix cookie issue not following device theme

* Revert "Working on fixing cookies"

This reverts commit a27eab5.

* Update cookie

trying to fix logic to update cookies correctly

* Fixed Cookie

believed to have fixed cookie issue, atleast it works on my local device

* Updating Default Logic

changed default logic to toggle follow theme if user gets rid of cookie or its first time loading site

* beep boop Github Action Master Branch CI Prettier hard at work

* Update ci.yaml

---------

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: Harry Tan <32246618+DarknessUnder@users.noreply.github.com>
Co-authored-by: dorian451 <112524240+dorian451@users.noreply.github.com>
Co-authored-by: Troy-bailan-King <97757033+Troy-bailan-King@users.noreply.github.com>
Co-authored-by: Matt Lammon <mlammon11@gmail.com>
Co-authored-by: Matt Lammon <91478985+mlammon11@users.noreply.github.com>
Co-authored-by: I'm a Github Action <beepboop@github.com>
Co-authored-by: charleschae12 <112535899+charleschae12@users.noreply.github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
@skyjune0824 skyjune0824 dismissed stale reviews from chavab0809 and mlammon11 via 3f49f69 April 7, 2023 20:10
@skyjune0824
Copy link
Contributor Author

Re-review requested because of master -> this merge to avoid some issues

@skyjune0824 skyjune0824 requested a review from dorian451 April 7, 2023 21:12
zhengw5
zhengw5 previously approved these changes Apr 7, 2023
Copy link

@zhengw5 zhengw5 left a comment

Choose a reason for hiding this comment

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

I just tested this function and everything works great. The feature is really practical and nice for the users and looks great.

Copy link
Contributor

@dorian451 dorian451 left a comment

Choose a reason for hiding this comment

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

i removed all of the unessesary files in your branch, otherwise looks good

@jonha1 jonha1 merged commit f7f6cf7 into master Apr 21, 2023
@jonha1 jonha1 deleted the animated_collapse branch April 21, 2023 21:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
front-end front-end related (usually in vue.js)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

10 participants