Project

General

Profile

Bug #8463

Continued issues with portfolio view on mobile device

Added by Matt Gold about 4 years ago. Updated about 4 years ago.

Status:
Resolved
Priority name:
Normal
Assignee:
Category name:
Public Portfolio
Target version:
Start date:
2017-08-03
Due date:
% Done:

0%

Estimated time:

Description

Hi Boone and Dan --

Following up on issue 8290, I just wanted to note that I am still seeing some issues:

1. Perhaps because I have a lot of text in my header section, there is something awkward that happens when I start to scroll down the page and the header snaps up -- basically, the text snaps up but the background does not. Screenshot attached.

2. The alignment on the section headers/text feels is still not working right. Please see attached.

image3.PNG (172 KB) image3.PNG text alignment issues - 2 Matt Gold, 2017-08-03 01:21 PM
image2.PNG (199 KB) image2.PNG text alignment issues - 1 Matt Gold, 2017-08-03 01:21 PM
image1 (1).PNG (379 KB) image1 (1).PNG header section snap issue Matt Gold, 2017-08-03 01:21 PM
Screenshot_2017-08-11_21-05-41.png (57.5 KB) Screenshot_2017-08-11_21-05-41.png Boone Gorges, 2017-08-11 10:07 PM

Related issues

Related to CUNY Academic Commons - Design/UX #8290: Problems with Commons profile on mobile viewResolved2017-06-19

Related to CUNY Academic Commons - Bug #8510: Social media icons crowded on mobile view of PortfolioResolved2017-08-13

History

#1 Updated by Matt Gold about 4 years ago

  • Related to Design/UX #8290: Problems with Commons profile on mobile view added

#2 Updated by Boone Gorges about 4 years ago

  • Assignee changed from Boone Gorges to Daniel Jones
  • Target version set to 1.11.9

Dan, could you please have a look?

#3 Updated by Boone Gorges about 4 years ago

I fixed the widget title overlapping in https://github.com/cuny-academic-commons/cac-advanced-profiles/commit/2200bfb0306edfb2e4466bc37a003d0ba5632df2. The columns now collapse on narrow windows. See screenshot.

As for the "snap" issue, it is indeed a problem having to do with a combination of the narrow window and the amount of text in the header. I made the calculation a bit more dynamic in https://github.com/cuny-academic-commons/cac-advanced-profiles/commit/bb5dd0fd861ea6903d5ae8f0c5328141b9a50d81 and it seems to be working much better. It's probably possible to tweak this more, but it's massively better than it was, so I'm going to tentatively mark the ticket resolved.

Thanks for the report and for the screenshots.

#4 Updated by Matt Gold about 4 years ago

Confirming that this is much improved and all better. thank you, Boone!!

(I have one minor note but will open a separate ticket)

#5 Updated by Matt Gold about 4 years ago

  • Related to Bug #8510: Social media icons crowded on mobile view of Portfolio added

Also available in: Atom PDF