Problem/Motivation
The discussion in #2120875: Remove breakpoint and picture module from core was resolved by closing that discussion but marking as needing to be revisited before D8 beta. A number of issues were identified that needed to be addressed in the Picture and Breakpoints modules. This issue is intended to track those and any new issues that arise before beta.
Update
Since D8.0 beta is here, this is all about followups now.
Open issues
#1836860: Picture doesn't work with AJAX callback
#1855412: Enable responsive_image.module by default for standard install profile
#1701116: Responsive Images UI/UX: Image styles
#1880606: Introduce a configuration UI for theme-based breakpoints
#1898442: responsive_image.module - Convert theme_ functions to Twig
#2123251: Improve DX of responsive images; convert theme functions to new #type element
#2220865: Add empty img element inside picture element
#2349461: Move fallback image style into the responsive image style entity
#2348255: Provide option to use srcset and/or sizes attributes on img tag instead of the picture element
#2349859: Responsive Image Mappings :: throws a fatal exception if no image style is set
#2260061: Responsive image module does not support sizes/picture polyfill 2.2
#2343351: Make picture polyfill optional
#2334387: UI changes to support current responsive image standards
#2372187: Add getDescription in Breakpoint module
Fixed/Closed issues
#2254071: [PP-1] Responsive images can't be linked to Content
#2254071: [PP-1] Responsive images can't be linked to Content
#1701112: Advanced responsive images UI/UX for breakpoints
#2203459: responsive_image mapping doesn't implement 'delete' function
#1937914: Special handling for setting the BreakpointGroup::breakpoints property during ConfigStorageController::importChange()
#1828606: Make BREAKPOINT resources inheritable
#1851018: Improve breakpoint configuration implementation.
#1883526: Decide on the picture polyfill to use
#2123225: Do not upscale by default
#2026319: Mis-named variables in picture_theme()
#2030653: Expand ResponsiveImageMapping with methods
#2078001: Modernize responsive_image.module forms
#2124377: Rename "Picture" module to "Responsive Image" module
#2132551: Picture module uses config keys with a dot
#2204159: (Responsive) Image styles do not add correct cache tags, nor do they invalidate cache tags upon flushing
#2211831: Removal of alt attribute from [picture] tag
#2219329: ResponsiveImageMapping::mappings and ResponsiveImageMapping::breakpointGroup properties have inconsistent return structure and type respectively
#2222431: Project name clean up after Picture module renamed Responsive Image
#2225677: Fix naming in ResponsiveImageMappingEntityTest following module rename from Picture to Responsive Image
#2227435: Remove noscript from picture element markup
#2243549: Text changes needed as follow-up to Menu UI module rename
#2245727: Add missing configuration schema in Breakpoint component
#1820512: Move Breakpoint module closer to meeting Drupal standards
#1945228: Repetitive config in breakpoint.module
#1945362: error messages on admin page for picture mappings
#1971174: BreakpointGroup::loadAllBreakpoints() is expensive
#1982818: Automatically sort breakpoints from small to large
#1999312: Add an 'empty image' option for responsive image
#2030585: Expand Breakpoint with methods
#2030587: Expand BreakpointGroup with methods
#2271529: Move breakpoint settings to theme and module *.breakpoints.yml files at root level
#2349789: Responsive Image Mappings are not listed with sqlite
#2267563: Uncaught SQLite failure breaks responsive images
#2315077: Using the "Link image to" feature of the picture formatter causes a fatal error
#2342439: Some test modules/themes missing "Testing" package assignment in their info files.
#2318801: Breakpoint Module: Fix documentation that refers to enabling/disabling of modules
#2030587: Expand BreakpointGroup with methods
Comments
Comment #1
dcrocks CreditAttribution: dcrocks commentedAdded new issue to list
Comment #2
dcrocks CreditAttribution: dcrocks commentedComment #3
dcrocks CreditAttribution: dcrocks commentedComment #4
dcrocks CreditAttribution: dcrocks commentedComment #5
dcrocks CreditAttribution: dcrocks commentedComment #6
dcrocks CreditAttribution: dcrocks commentedComment #7
dcrocks CreditAttribution: dcrocks commentedComment #8
dcrocks CreditAttribution: dcrocks commentedComment #9
dcrocks CreditAttribution: dcrocks commentedComment #10
dcrocks CreditAttribution: dcrocks commentedComment #11
dcrocks CreditAttribution: dcrocks commentedComment #12
dcrocks CreditAttribution: dcrocks commentedComment #13
dcrocks CreditAttribution: dcrocks commentedSome of these added because they hit breakpoint/responsive_image code. Some should be reviewed to see if they are still relevant.
Comment #14
dcrocks CreditAttribution: dcrocks commentedNot sure how to properly handle 'related' issues. These touch breakpoint/responsive_image code bur in a minor way. Should I just track them in a list?
Comment #15
dcrocks CreditAttribution: dcrocks commentedComment #16
dcrocks CreditAttribution: dcrocks commentedComment #17
dcrocks CreditAttribution: dcrocks commentedComment #18
attiks CreditAttribution: attiks commentedJust a heads up #2271529: Move breakpoint settings to theme and module *.breakpoints.yml files at root level will simplify everything of breakpoints and will close a lot of issues,
Comment #19
attiks CreditAttribution: attiks commented#2271529: Move breakpoint settings to theme and module *.breakpoints.yml files at root level got committed, I closed the other ones
Comment #20
RainbowArrayComment #21
RainbowArrayComment #22
RainbowArrayOnce we take care of #2260061: Responsive image module does not support sizes/picture polyfill 2.2, a lot of the postponed issues will get unblocked or will no longer be relevant.
Comment #23
RainbowArrayComment #24
dcrocks CreditAttribution: dcrocks commentedUpdates
Comment #25
dcrocks CreditAttribution: dcrocks commentedComment #26
dcrocks CreditAttribution: dcrocks commentedComment #27
dcrocks CreditAttribution: dcrocks commentedComment #28
dcrocks CreditAttribution: dcrocks commentedComment #29
dcrocks CreditAttribution: dcrocks commentedComment #30
Jeff Burnz CreditAttribution: Jeff Burnz commentedI have made a feature request for Breakpoints: #2372187: Add getDescription in Breakpoint module
Comment #31
dcrocks CreditAttribution: dcrocks commentedComment #32
dcrocks CreditAttribution: dcrocks commented