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

Enhanced Profiles Page #350

Merged
merged 1 commit into from
Jan 15, 2022
Merged

Enhanced Profiles Page #350

merged 1 commit into from
Jan 15, 2022

Conversation

S-kus
Copy link
Contributor

@S-kus S-kus commented Jan 15, 2022

I fixed the problem in this. As shown in this Video.

  • Made page responsive.
  • Now title and last updated date are also showing.
  • Return to main page tag also present.
  • I removed extra padding and made fonts-size similar to other nav-links and fonts.

@quozl
Copy link
Contributor

quozl commented Jan 15, 2022

Added review changes. As far as I can see, the profile layout wasn't used; can someone else please check for me?

@srevinsaju
Copy link
Member

srevinsaju commented Jan 15, 2022

As far as I can see, the profile layout wasn't used

Agreed, I don't see it being used anywhere too. +1 for remove

Bullet points apepared in navbar with 1ae8e43.

* updated 'page title' and 'return to main-page',

* made page responsive to geometry changes,

* add title and last updated,

* add return to main page link,

* remove extra padding,

* use similar font sizes,

* remove unused profile layout.
@quozl quozl merged commit a41d135 into sugarlabs:master Jan 15, 2022
@quozl
Copy link
Contributor

quozl commented Jan 15, 2022

Please use our guide to making commits and make sure the commit messages have the detail that you've been adding to the pull request.

@S-kus
Copy link
Contributor Author

S-kus commented Jan 16, 2022

Sorry @quozl, I will be more careful next time.

can someone else please check for me?

In navbar in the "Join Us" dropdown, it's referred link present as "contributors" and also in the "SugarLabs" section in the footer named as "contributors".

@quozl
Copy link
Contributor

quozl commented Jan 16, 2022

Yes, interesting.

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.

3 participants