CUNY Academic Commons

- Members section: the names on the "From your campus" side need to be aligned https://share.getcloudapp.com/7KuqxgPQ (looks great on the logged out)

Fixed in https://github.com/cuny-academic-commons/cac/commit/12bb6d067047db52e2b2dd5558c94bf246a08fd2. This is available on cdev for testing.

Logged In Homepage:
- Under Groups, when the title gets to be two lines the title/time does not align to the top of the thumbnail https://share.getcloudapp.com/9ZuDWrEP
- Same with sites https://share.getcloudapp.com/6quED72y
- Members section: section title and "discover more members" - not centered in the background. The background seems thicker than groups and sites https://share.getcloudapp.com/rRuNnX7O

Logged Out:
- active groups have the same alignment issue for 2 lines as above

I've aligned all list-item titles to the top in https://github.com/cuny-academic-commons/cac/commit/97d96d9845168569f9620dbf6d5d1be45129d854. This is also available on cdev for testing.

- When I load the page, the campus groups and sites seem to be "lazy" in their loading ... they need me to scroll to load it seems. Could they load on pageload?

I coded the suggestions blocks to lazyload intentionally. See https://redmine.gc.cuny.edu/issues/14181#note-7. This is done to speed up the initial page load time. If others feel this is not a good idea, I can look into refactoring the code to load as part of the page.

Mobile:
- when you get to the scrolling sections, the next item should go to the edge of the screen:https://share.getcloudapp.com/WnuqA4k8

If we remove the right margin here, if you scroll all the way to the right, there will not be any margin. See https://i.postimg.cc/NMk8N1hT/scroll-no-right-margin.png. Is that okay?

Thanks for jumping in on these, Ray.

If we remove the right margin here, if you scroll all the way to the right, there will not be any margin. See https://i.postimg.cc/NMk8N1hT/scroll-no-right-margin.png. Is that okay?

I was able to juggle the markup so that this isn't necessary. I've made the change in https://github.com/cuny-academic-commons/cac/commit/b92fd1a3f7f9aa560f3ce1930969b6ee013621e5 and it's ready to test on cdev. (Be sure to hard-refresh or clear browser cache.)

I coded the suggestions blocks to lazyload intentionally. See https://redmine.gc.cuny.edu/issues/14181#note-7. This is done to speed up the initial page load time. If others feel this is not a good idea, I can look into refactoring the code to load as part of the page.

How about just setting the scroll threshold a bit higher so that no one notices? Maybe 100px before it appears on page, or something like that? IMO the lazyload is a nice performance enhancement (especially on mobile, where scroll lengths are long and bandwidth could be an issue) so I'd prefer not to scrap it.

Thanks for these notes, Sara. On the lazy loading, I agree if we change the scroll threshold so the loading is just a bit more hidden, it would be nice to keep it for performance.

I was able to juggle the markup so that this isn't necessary. I've made the change in https://github.com/cuny-academic-commons/cac/commit/b92fd1a3f7f9aa560f3ce1930969b6ee013621e5 and it's ready to test on cdev. (Be sure to hard-refresh or clear browser cache.)

Unfortunately, this commit broke the two-column layout for the logged-in homepage. I reverted this commit and found an easier fix: https://github.com/cuny-academic-commons/cac/commit/90764a2db4ad22835fb93244f453e66e311a7c76. This is ready for testing on cdev.

How about just setting the scroll threshold a bit higher so that no one notices? Maybe 100px before it appears on page, or something like that?

I've added a 300px threshold. See https://github.com/cuny-academic-commons/cac/commit/3d7eeb9a8d187ceb8db966704359de4ae816a44b. This is also ready for testing on cdev. You might need to do a hard refresh so the cac-suggestions JS is updated. When testing, if you're scrolling slow, you will not notice the loading, but if you're scrolling fast, you might. Let me know if we want to bump up the threshold even more.

Thanks Boone!