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

Part 2 UI bug-fixes/enhancements #166

Merged
merged 16 commits into from
Nov 29, 2023
Merged

Part 2 UI bug-fixes/enhancements #166

merged 16 commits into from
Nov 29, 2023

Conversation

rmgutierrez
Copy link
Contributor

@rmgutierrez rmgutierrez commented Nov 29, 2023

Major

Problem: Found out that unlisted posts are still sent to followers because they are technically PUBLIC posts, but unlisted was never checked.

  • Fix: changed visibility === sharetype.PUBLIC to visibility === sharetype.PUBLIC && !unlisted when sending PUBLIC posts to follower inbox.

Problem: Author can visit another author's profile page and see their unlisted posts.

  • Fix: changed post.visibility === sharetype.PUBLIC to visibility === sharetype.PUBLIC && post.unlisted == false when filtering posts to show the author.

Not a problem but I added handleAvatarClick to both PostPage and PostsList to navigate to author's profile page (More than a UI fix/enhancement)

Minor

  • PostPage

    • Fixed the loading animation (it used to appear at the bottom of the screen)
    • Minimized the chip that shows whether remote or local
    • Added click to avatar to navigate to author's profile page
  • PostsList

    • Also minimized the chip that shows whether remote or local
    • Also added click to avatar to navigate to author's profile page
  • MakePostModal

    • Fixed the "Select a follower" input when making private posts
      • Modified its size to be more consistent with the other inputs
  • Loading

    • Changed the colour of the circular progress to grey, as well as made it a bit thicker
  • ProfilePage/ProfileTabs

    • Update github link format
      • Removed "https://" when rendered, removed underline, changed font weight
    • Added loading animation to profile page
    • Also added loading animations to both posts list and followers list within profile page
  • SharePostModal

    • Added the text "You have no followers..." if user presses the share button and they don't have any followers
  • AppBar

    • Made the sign-out button a lighter colour
    • Aligned "distributed" with the buttons on the LeftNavBar
      • Taking inspiration from Twitter (this can be changed, I'm really not sure which one looks better)
      • (My fault) Had a bug where the entire left side of "distributed" linked to home page, now fixed
  • LeftNavBar

    • Added a max width to the post button
      • It grew longer for higher resolution devices
  • GithubEventList

    • Added primary color to github logo and text
    • Prevent bug where the list would shift to the right when the modal is opened
      • Positioned from the left instead of the right

Copy link
Contributor

@kaynzhel kaynzhel left a comment

Choose a reason for hiding this comment

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

lgtm

@rmgutierrez rmgutierrez merged commit f690a63 into main Nov 29, 2023
1 check passed
@rmgutierrez rmgutierrez deleted the part-2-ui-enhancement branch November 29, 2023 18:48
@kaynzhel kaynzhel mentioned this pull request Nov 29, 2023
kaynzhel added a commit that referenced this pull request Nov 29, 2023
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.

UI: do a pass through on general pages and see if there are any design improvements that can be made
2 participants