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

Improved UI in LookupConditions #39

Merged
merged 2 commits into from
Oct 29, 2023
Merged

Improved UI in LookupConditions #39

merged 2 commits into from
Oct 29, 2023

Conversation

hidacow
Copy link
Contributor

@hidacow hidacow commented Oct 14, 2023

  • If user already selected a major and changes the college, the selected major will not change even it does not exist in the newly selected college. This has been resolved by binding a function to reset major to default, which is 全部, when college changes.
  • Enabled search in a-select to make it easier to filter through a long list of colleges, majors and properties.

Copy link
Member

@BH4HPA BH4HPA left a comment

Choose a reason for hiding this comment

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

Nice improvement, thanks for your contribution!

@BH4HPA BH4HPA merged commit 45a5584 into shuosc:main Oct 29, 2023
1 check passed
@BH4HPA
Copy link
Member

BH4HPA commented Oct 29, 2023

@hidacow Please review your changes at https://pre.xk.shuosc.com/ which was deployed. Once confirmed, I'll deploy it on the main domain.

@hidacow
Copy link
Contributor Author

hidacow commented Oct 30, 2023

It looks good! Deploy it to production please.

@BH4HPA
Copy link
Member

BH4HPA commented Oct 30, 2023

@hidacow Great! 🎉 Thank you for confirming. I have now completed the deployment. If you encounter any issues or have further feedback, please do not hesitate to let me know. 😊

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.

2 participants