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

[ZA] Simplify province pages #2475

Open
zarino opened this issue Sep 6, 2018 · 6 comments
Open

[ZA] Simplify province pages #2475

zarino opened this issue Sep 6, 2018 · 6 comments

Comments

@zarino
Copy link
Member

zarino commented Sep 6, 2018

Based on comments from Gaile and Rashaad about "province pages" like https://www.pa.org.za/place/eastern-cape/

screenshot_2018-09-06 people s assembly

  • The “People / Constituency Offices” tabs are broken (they don‘t look like tabs at all!) – Megan suggests dropping the “tabs” concept entirely, and just have the people list/accordion always visible, meaning we don’t need a “people” tab at all.
    • Effectively we’d be combining /place/<name> and /place/<name>/places so that people and places are always included on the same page all the time (but folded away nicely in the accordion, as currently).
    • Need to check performance implications of this. Will the server be able to render both sets of data quick enough?
    • Will probably need some developer time, for combining the Django views.
  • It’s not made very clear that—for example—“Members of the Provincial Legislature (63)” contains past members as well as present members – Megan suggests breaking past/present out into separate accordion sections?
  • They want to rename those accordion links so they’re no longer all “Members of…”. New wording:
    • Members of the Provincial Legislature
    • National Assembly members from provincial list
    • Permanent Delegates to National Council of Provinces
  • Remove the “Expand all subsections” link and the “There are 121 people related to…” sentence.
  • Remove grey lines beside Constituency Office list items.
    • I think they’re there because the template is trying—and failing—to print out the people related to that location?
@zarino
Copy link
Member Author

zarino commented Oct 12, 2018

#2494 has done some work towards tidying up the bottom half of this page, by fixing the tab styling and removing the summary row:

screen shot 2018-10-12 at 09 42 10

Still to do (as above):

  • Show both people and offices on the same page
  • Split out historical memberships
  • Rename the memberships
  • Remove the grey lines beside office list items

@chrismytton
Copy link
Member

PMG have asked if it's possible to get rid of the whitespace to the right of the Google Map?

@zarino
Copy link
Member Author

zarino commented Oct 18, 2018

PMG have asked if it's possible to get rid of the whitespace to the right of the Google Map?

It is annoyingly difficult to do. Under the hood, the place pages are use the same layout at the person pages, where the person’s photo is floated left, name and details floated right, and then the rest of the page content continues below them.

The layout is less noticeable on person pages, because the person’s name is followed by one or more contact buttons, which fill the vertical space to the side of the photo, and then the photo itself is followed by a sidebar of contact details, again filling up space.

The only way to remove this space from the place pages would be to have them no longer inherit the person page layout. It’ll be a fair chunk of work, but maybe worth it? Could be something to discuss on our next monthly call, @GeorgieBurr?

@chrismytton chrismytton self-assigned this Oct 18, 2018
@GeorgieBurr
Copy link

GeorgieBurr commented Oct 18, 2018

The call is in another 2/3 weeks from now... do you think it could wait until the call to be discussed? I have added it to the agenda :)

@GeorgieBurr
Copy link

GeorgieBurr commented Nov 20, 2018

Action break up into smaller tickets:

@chrismytton
Copy link
Member

  • They want to rename those accordion links so they’re no longer all “Members of…”. New wording:
    • Members of the Provincial Legislature
    • National Assembly members from provincial list
    • Permanent Delegates to National Council of Provinces

This has been done in #2544.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants