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.

Sign up for GitHub

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

Jump to bottom

[UX/UI] Mosaic tile height rendering issue #247

Open
UcDust opened this issue Nov 18, 2024 · 2 comments
Open

[UX/UI] Mosaic tile height rendering issue #247

UcDust opened this issue Nov 18, 2024 · 2 comments
Assignees
@UcDust
Labels
bug Something isn't working pre-release High priority features of fixes that need to be in the initial release UX / UI

Comments

@UcDust
Copy link
Contributor

UcDust commented Nov 18, 2024

Describe the bug
Noticed on android chrome tablet/phones, seems to sometimes not render the height correctly on nav from the browse page, as well as adding/removing various filters.
@UcDust UcDust added bug Something isn't working UX / UI pre-release High priority features of fixes that need to be in the initial release labels Nov 18, 2024
@UcDust UcDust self-assigned this Nov 18, 2024
UcDust added a commit that referenced this issue Nov 21, 2024
@UcDust
fix for mosaic tiles not rendering with correct heights #247
687f6f6
jrmerz pushed a commit that referenced this issue Nov 26, 2024
@UcDust
[BUG/FEAT] os team testing fixes/changes (#254)
99c0bfc 
* bookreader should default to single page if < 800px width #245

* fix header z-index dropdown issue in firefox #249

* update typeahead with search icon, fix padding around search filters for scrollbar overlap #251

* small style change to input/margin after chatting with Kimmy #251

* fix for main content clipping issue covering footer on mobile #243

* scroll to top of item page on ark/doi link click #253

* fix for tif image load breaking render in ff #252

* update main content minheight so it doesnt exceed shorter content (browse page, browse formats, etc) #243

* fix for mosaic tiles not rendering with correct heights #247

* bugfix for browse page heights #243

* change fast links to be search links #257

* update mailto link to just go to library profile #260

* remove margin-top on item description markdown element #263

* clip bottom of creator images #261

* style updates for mobile/shrink layout on home page #269

* bookreader render issue fix when position not in order/start at position 1 #256 #258

* fix for selected image thumbnail with imagelist #268

* fix for refresh page showing correct selected thumbnail #268

* bugfix for #268

* add fullscreen control to video #259

* fix download label/button heights to scale better in different resolutions #246

* update citation dropdown to be left aligned, style fix for toast copy message #248

* date filter styles #262
@UcDust UcDust added the qa/qc In testing stage label Nov 26, 2024
@UcDust
Copy link
Contributor Author

UcDust commented Nov 26, 2024

@wrenaria this should be ready to test on dev.

@wrenaria
Copy link

wrenaria commented Dec 3, 2024

Still having trouble with mosaics showing correctly

MacOS Chrome example 1:

Screenshot 2024-12-03 at 9 15 56 AM

MacOS Chrome example 2:

Screenshot 2024-12-03 at 9 15 40 AM

iOS:

@wrenaria wrenaria removed the qa/qc In testing stage label Dec 3, 2024
UcDust added a commit that referenced this issue Dec 9, 2024
@UcDust
fix for mosaic intermittent render issue not calc img heights correctly
5836a75 
#247
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@UcDust UcDust

Labels
bug Something isn't working pre-release High priority features of fixes that need to be in the initial release UX / UI
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@wrenaria @UcDust