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

Sidebar UI #701

Merged
merged 9 commits into from
Feb 10, 2021
Merged

Sidebar UI #701

merged 9 commits into from
Feb 10, 2021

Conversation

joshslee
Copy link
Contributor

@joshslee joshslee commented Feb 9, 2021

Features

  1. Update sidebar UI for /hubs and /leaderboard page to match home page UI.
  • Add auto select on scroll for hub page for categories
  1. Fix scroll behavior bug for paper tabs on paper page.
  2. Update paper tab label to match section title.
  3. Update placeholders for leaderboard page and sections.
  4. Fix overflow bug on leaderboard user/author card.

Hubs_on_Researchhub

ResearchHub_Leaderboards

ResearchHub___Open_Science_Community

Six-month_Neurological_and_Psychiatric_Outcomes_in_236_379_Survivors_of_COVID-19

@joshslee joshslee added bug Something isn't working enhancement New feature or request labels Feb 9, 2021
@joshslee joshslee self-assigned this Feb 9, 2021
lightninglu10
lightninglu10 suggested changes Feb 10, 2021
View reviewed changes
Copy link
Contributor

@lightninglu10 lightninglu10 left a comment

Choose a reason for hiding this comment

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

remove extraneous pieces of code

@joshslee joshslee merged commit e89ae0c into master Feb 10, 2021
@joshslee joshslee deleted the sidebar-ui branch February 10, 2021 18:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@lightninglu10 lightninglu10 lightninglu10 requested changes

Assignees
No one assigned
Labels
bug Something isn't working enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@joshslee @lightninglu10