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

Fix: Make links page responsive on mobile view #10033

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

sohail-313
Copy link

Fixes Issue

Closes #9907

Changes proposed

  • Implemented responsive design for the links page on mobile view.

Check List (Check all the applicable boxes)

  • My code follows the code style of this project.
  • My change requires changes to the documentation.
  • I have updated the documentation accordingly.
  • All new and existing tests passed.
  • This PR does not contain plagiarized content.
  • The title of my pull request is a short description of the requested changes.

Video Recording

links-responsive.mov

Note to reviewers

No specific notes for reviewers at the moment. The changes are straightforward and align with the outlined bugfix for the responsive issue. If you have any questions or suggestions, please feel free to let me know.

@github-actions github-actions bot added the issue linked Pull Request has issue linked label Dec 27, 2023
@sohail-313 sohail-313 changed the title Fix: Make links page responsive on mobile view (#9907) Fix: Make links page responsive on mobile view Dec 27, 2023
Copy link
Contributor

@amandamartin-dev amandamartin-dev left a comment

Choose a reason for hiding this comment

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

Hello! This looks good on the account page, but unfortunately is affecting the public facing profile. On that page the link text should remain left aligned (versus centered).

Please adjust so these changes only affect the account management view.

@sohail-313
Copy link
Author

Hello! This looks good on the account page, but unfortunately is affecting the public facing profile. On that page the link text should remain left aligned (versus centered).

Please adjust so these changes only affect the account management view.

Can you please share screenshot

@eddiejaoude
Copy link
Member

The other places links component is used does not look right. I think it might be better to hide the extra info (badges) on smaller screens and truncate the text

Screenshot 2024-01-15 at 09 26 31

@sohail-313
Copy link
Author

The other places links component is used does not look right. I think it might be better to hide the extra info (badges) on smaller screens and truncate the text

Screenshot 2024-01-15 at 09 26 31

Yes, it can be done. However, what if the user wants to edit those links?

@eddiejaoude
Copy link
Member

what if the user wants to edit those links?

don't hide the edit button, just the extra info, which they can then see and edit when they edit the link

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
issue linked Pull Request has issue linked
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[BUG] links page in not responsive on mobile view
3 participants