Image ratio is off in lots of places #29
Labels
No labels
bug
discussion
enhancement
No milestone
No project
No assignees
1 participant
Notifications
Due date
No due date set.
Dependencies
No dependencies set
Reference
edufeed/edufeed-app#29
Loading…
Reference in a new issue
No description provided.
Delete branch "%!s()"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
I noticed that we display images in lots of places with bad ratio, e.g.
naddr1qvzqqqrukvpzqna968zp8c45tcgdgzln2c4twqd9xvfqdc6eey964c8fn07kcmjpqqjxzwpex5ukydrp95ekxepc956r2e3e95unvc3k95mn2wrpxyerzvpnxycxv252m6yand this happens in other places as well. we should have a concept regarding how much space images may take and how to render those that dont fit nicley in (with correct ratio and not cutting off parts of the images)
Concept + first implementation on branch
fix/issue-29-hero-image-ratio.Concept:
Implemented: new shared
HeroImagecomponent (unit-tested), applied to the calendar event detail view (the reported case — the EduFeed logo now shows complete), the event details modal and the article view. Verified with the naddr from the report.If there are further specific places with bad ratios beyond these three detail views, list them here and I will migrate them to the same component.
Confirmed by Steffen, merged to dev — including the follow-up from testing: event-bar labels now switch between black and white based on the background color luminance (>=4.5:1 contrast against every generated author color), so dark blue/purple bars are readable. Branch fix/event-bar-text-contrast, also merged to dev.