-
Notifications
You must be signed in to change notification settings - Fork 0
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
Collapse abstracts #88
base: main
Are you sure you want to change the base?
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Source changes look good, @ntlhui to deploy local to review build
One big thing is maybe styling could use work |
Removed block that says abstract, clicking the title of the paper now opens abstract Removed unnecessary padding Note: not intuitive potentially, should think of UI design here
The Maybe a better styling might be an |
Interesting, let me take another look and maybe come up with a few ideas |
Does this consider that hovering over the text makes the textbox highlighted and underlined? edit: I agree the + is not immediately obvious. I think I'm going to bold it and make it bigger |
Pull request was converted to draft
No description provided.