#31004 closed enhancement (duplicate)
Support srcset in avatars.
Reported by: | georgestephanis | Owned by: | |
---|---|---|---|
Milestone: | Priority: | normal | |
Severity: | normal | Version: | 4.2 |
Component: | General | Keywords: | needs-refresh reporter-feedback |
Focuses: | Cc: |
Attachments (1)
Change History (8)
This ticket was mentioned in Slack in #feature-respimg by georgestephanis. View the logs.
10 years ago
#4
@
10 years ago
The biggest problem I see here is that the size and url args were both made filterable already. This patch assumes that the size arg corresponds to the 1x size for the url arg, which is no longer a valid assumption.
#31469 might be a step in the right direction, and even that might need to be expanded to keep all the different variables separated.
#5
@
9 years ago
- Keywords needs-refresh reporter-feedback added; has-patch removed
Do we still need to do anything here after [31721]?
Note: See
TracTickets for help on using
tickets.
I'm definitely in favour of adding
srcset
support, though I have a couple of reservations about this patch.I'd like to see this land in the feature plugin first. If there are any extra filters required, we can probably add them in (either temporary or permanent, as needed).