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

Add portrait orientation card display for AZ Person #3757

Open
akslay opened this issue Oct 7, 2024 · 2 comments
Open

Add portrait orientation card display for AZ Person #3757

akslay opened this issue Oct 7, 2024 · 2 comments
Assignees
Labels
enhancement New feature or request

Comments

@akslay
Copy link
Contributor

akslay commented Oct 7, 2024

Motivation

Many faculty/staff have portrait orientation professional headshots taken for their department's staff listings. Portrait orientation headshots generally result in zoomed in or cut off faces in the AZ Person card grid displays.

Proposed Resolution

Modifying the default view display "Grid Page" is undesirable as it would impact existing sites, some of which may have opted for landscape orientation photos of staff to address the aspect ratio issue. Recommend adding a new display mode and view display so that sites can benefit without adversely impacting other sites.

  • Add "Card - Portrait" crop type
    (I am connecting with our UX team for a recommended aspect ratio, will update ticket with ratio)
  • Add "Card - Portrait" image style
  • Add a "Card - Portrait" display mode and configure it for AZ Person (duplicate the "Card" display mode and swap out image style used)
  • Add a "Grid Page - Portrait" view display to AZ Person view (duplicate Grid Page)
@akslay akslay added the enhancement New feature or request label Oct 7, 2024
@akslay akslay self-assigned this Oct 7, 2024
@akslay akslay changed the title Add a portrait orientation image style & view display for AZ Person card displays Add portrait orientation image style & view display for AZ Person card displays Oct 8, 2024
@akslay akslay changed the title Add portrait orientation image style & view display for AZ Person card displays Add portrait orientation card display for AZ Person Oct 8, 2024
@joshuasosa
Copy link
Contributor

I would suggest changing the default for new sites to portrait. I don't think card should have been set as the default. I would even expect most existing sites that use the default view display could benefit from a change to portrait instead of letting heads get cut off all over the place.

Examples where changing to portrait could be an improvement:
https://kalfaa.lpl.arizona.edu/people
https://it.arizona.edu/about-us/meet-it-leadership
https://president.arizona.edu/leadership-team
https://law.arizona.edu/about/leadership
https://drc.arizona.edu/people

@akslay
Copy link
Contributor Author

akslay commented Oct 9, 2024

Having it be default on new sites sounds reasonable to me.

Though if we did end up making it default on all sites, this site would be a good one to test the change on since they have landscape orientation staff photos: https://rec.arizona.edu/staff

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants