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

UI Improvements Planning Issue : People Section #5707

Closed
6 of 8 tasks
CleverFool77 opened this issue May 11, 2019 · 43 comments · Fixed by #5740, #5754, #5757, #5765 or #5806
Closed
6 of 8 tasks

UI Improvements Planning Issue : People Section #5707

CleverFool77 opened this issue May 11, 2019 · 43 comments · Fixed by #5740, #5754, #5757, #5765 or #5806
Labels
design issue requires more design work and discussion (i.e. mockups and sketches) outreachy summer-of-code

Comments

@CleverFool77
Copy link
Member

CleverFool77 commented May 11, 2019

Description

The basic Card UI of people section is already on halfway.
Now the further improvements are left.
As we were told during proposal that we should start off with easier ones first. And they should be given beginning of time in timeline. So we will start off with people section first which is the easiest.
The final UI for the people's page :

Screen Shot 2019-05-19 at 2 41 35 AM

@gautamig54 We will add checklists here

card ui of users

location

search by

  • Add search by various places option below maps.

others

  • Make the map sticky.
  • position map on the top instead of bottom, in the mobile view @CleverFool77
  • Fix overflow of tags @CleverFool77
@CleverFool77 CleverFool77 changed the title UI Imporvements Planning Issue : People Section UI Improvements Planning Issue : People Section May 11, 2019
@CleverFool77 CleverFool77 added the design issue requires more design work and discussion (i.e. mockups and sketches) label May 11, 2019
@gautamig54
Copy link
Contributor

@CleverFool77 Let's first start with the questions page as was mentioned by @jywarren.
Also, The six pages mentioned earlier only need to be redesigned?
We have the blog page remaining as well. The task was divided amongst lot of us, but since we are suppose to work towards UI implementation, can we complete the blog2 page as well?

@CleverFool77
Copy link
Member Author

Hi @gautamig54 Actually, People section has most of the part done.
So, I was thinking of completing the section one by one.
But yeah if it was suggested by @jywarren , We can go with questions page first.
I'll open a new issue for questions page and break them into parts.
regarding blog2, I'll go and look into the matter that How much parts is already done.
I have some project evaluations and other pending stuff of this semester, So I won't fully active till 18th of May. Thanks for the patience.
And @gautamig54 It would be great if you could check out the other issue #5708 that I've opened. I have added some points in checklists.
Once again. Thanks !!!

@jywarren
Copy link
Member

Hi, all! I was thinking, based on the new style guide that is rapidly coming to completion, what if we focused on 2 things: first, looking for any remaining Bootstrap 4 issues to fix, and 2nd, implementing the "content card" sections of the style guide and moving them into a common area like app/views/content/_person.html.erb, app/views/content/_post.html.erb and app/views/content/_topic.html.erb? Then they can be re-used all across the site.

@CleverFool77
Copy link
Member Author

CleverFool77 commented May 15, 2019

Hi @jywarren
This looks like great Idea.
But I wanted to ask that How should I divide the work in todos and milestones which was mentioned by siddharth in one of the issues of planning.
I'm bit confused regarding the process.

@jywarren
Copy link
Member

Well, it sounded like people wanted to try using Projects, or planning issues with checkboxes and lists. You could do either. The main thing is to move a lot of the items from your proposals into GitHub somewhere so they can be updated as you go and boxes checked and issues closed bit by bit. Milestones are another way to manage this. I don't really have a strong preference as long as each team can coordinate between them, and others across the community can keep up to date on your overall progress. Make sense?

@gautamig54
Copy link
Contributor

@CleverFool77 You are right! Completing people's sections seems like a great start. I have updated the issue. Also, I was thinking if we could add a search feature here. It seems relevant and necessary to add search feature. What do you think about it?

@CleverFool77
Copy link
Member Author

Hi @gautamig54
Which search are you talking about ?
I'm not clear regarding that.
Thanks !!!!!

@gautamig54
Copy link
Contributor

Search feature as in where anyone can search for a member on the page. Like we have a search feature on tags page. You get it @CleverFool77 ??

@gautamig54
Copy link
Contributor

gautamig54 commented May 19, 2019

@CleverFool77 I also see a change in the margins due to which there is a change in card design from vertical to horizontal.

Screen Shot 2019-05-19 at 4 45 38 PM

Screen Shot 2019-05-19 at 4 45 47 PM

What do think about this? Should we change that as well?

@CleverFool77
Copy link
Member Author

Let's discuss regarding search feature with @jywarren .
@gautamig54 Actually regarding card design. There were two designs. So I went for vertical for now.
But now as we are adding new features. I'll fix it. I have added it in checklist.

@gautamig54
Copy link
Contributor

gautamig54 commented May 19, 2019

@CleverFool77 So you will be changing the margins as well??
I will get started with adding the user's location.

@CleverFool77
Copy link
Member Author

Hi @gautamig54
I guess you have to wait a bit as first #5740 need to get reviewed and merged. How about doing the work in Questions Page like me.
As I told you earlier. I'm taking two issues sideways. As some parts or portions of some issues are dependent on each other and it might make us to stop for a while as we are unable to move further.
Thanks !!!

@gautamig54
Copy link
Contributor

Yes @CleverFool77. Even I will follow the same strategy and work on two tasks simultaneously. I will implement the Add location option as it is not related to the cards design.

@CleverFool77
Copy link
Member Author

Hi @gautamig54
yes, you can start off with location as it doesn't seem related.
thanks !!!

@gautamig54
Copy link
Contributor

Also, alignment of the left and right column is required.

drawisland

@CleverFool77
Copy link
Member Author

Thanks for mentioning @gautamig54 I guess people text went out of column grid. That's why it wasn't algined.
I'll fix it.
Thanks !!

@CleverFool77
Copy link
Member Author

CleverFool77 commented May 21, 2019

Oh I just went through it and realised that it's due to margin-top in map. I'll fix it. Thanks !!!

Fixed UI

Screenshot from 2019-05-21 16-30-09

I've updated it in horizontal cards PR #5740

@gautamig54
Copy link
Contributor

gautamig54 commented May 30, 2019

@jywarren I was going through the style guide and found a new design for the /people page.

Screen Shot 2019-05-30 at 6 04 37 AM

Is this the updated design for the people's page?
cc: @CleverFool77

@CleverFool77
Copy link
Member Author

@CleverFool77 the tags are exceeding the card boundaries. Can you check this.

Screen Shot 2019-05-30 at 3 34 40 AM

Hmmm @gautamig54 @jywarren Any suggestions How should we deal with it ?

  • Restricting tagname maybe ?
  • add ellipses at overflow maybe ?

@jywarren
Copy link
Member

jywarren commented May 30, 2019 via email

@CleverFool77
Copy link
Member Author

Hi @jywarren
This thing about small map which expands looks cool idea. Because we usually don't prefer a big map spanning over the huge area on page rather a small open which we can span when we specifically want to look over it.
I guess we should implement this

@gautamig54
Copy link
Contributor

Yes @jywarren even I think smaller map is a better idea. It takes up half the page, due to which the space below the map is wasted. It will help optimising the page. We can definitely go with this!

@gautamig54
Copy link
Contributor

@CleverFool77 I played around a bit with font-size and padding. How do the cards look now?

Screen Shot 2019-06-05 at 2 15 53 AM

Screen Shot 2019-06-05 at 2 16 02 AM

@jywarren
Copy link
Member

jywarren commented Jun 4, 2019

ok, glad we have some consensus on small map design! One question I had was, when you expand it, should it open to cover the whole page, or should it direct you to a new page at, for example, publiclab.org/maps (although at the location and with the same layers showing?)

@gautamig54
Copy link
Contributor

How about a modal, not covering the entire page?

@jywarren
Copy link
Member

jywarren commented Jun 4, 2019 via email

@gautamig54
Copy link
Contributor

gautamig54 commented Jun 6, 2019

How about something like this @jywarren @CleverFool77
The modal will open up on clicking the map. People can zoom in and out the map on the modal itself.

Screen Shot 2019-06-07 at 1 18 25 AM

@gautamig54
Copy link
Contributor

@CleverFool77 regarding the exceeding of the tags from the card, I think adding ellipsis at overflow is a better option than restricting the tag name.

@jywarren
Copy link
Member

jywarren commented Jun 6, 2019 via email

@gautamig54
Copy link
Contributor

Great work with the tags overflow @CleverFool77. It will be good if you could display the the entire tag on hover over the tags. What say?

@stale
Copy link

stale bot commented Oct 7, 2020

Hi 😄, this issue has been automatically marked as stale because it has not had recent activity. Don't worry you can continue to work on this and ask @publiclab/reviewers to add "work in progress" label 🎉 . Otherwise, it will be closed if no further activity occurs in 5 days -- but you can always re-open it if you like! 💯 Thank you for your contributions 🙌 🎈.

@stale stale bot added the stale label Oct 7, 2020
@jywarren jywarren removed the stale label Oct 8, 2020
@jywarren
Copy link
Member

jywarren commented Mar 9, 2021

This is basically done! Thanks, all!

@jywarren jywarren closed this as completed Mar 9, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment