Closed (fixed)
Project:
Responsive images and styles
Version:
7.x-2.x-dev
Component:
Code
Priority:
Normal
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
24 Nov 2012 at 19:35 UTC
Updated:
17 Jan 2013 at 10:30 UTC
1) create a breakpoint with a rule like this:
all and (min-width: 980px) and (min-device-width: 980px), all and (max-device-width: 1024px) and (min-width: 1024px) and (orientation:landscape)
2) go to /admin/config/media/resp_img/create_style and create a new style, set base name to "mobile"
At this point the new style name is
mobileall and (max-width: 740px) and (max-device-width: 740px)
If you then go to edit this style, you can't, because with a messed up style name like this, image style form at admin/config/media/image-styles/edit/% won't validate...
Please only use lowercase alphanumeric characters, underscores (_), and hyphens (-) for style names.
Comments
Comment #1
tmsimont commentedupdated to most recent dev, still an issue.
Comment #2
attiks commentedWe'll backport the fix from picture module.
Comment #3
merch commentedThere appear to be two forms available to create styles
#1 has used admin/config/media/resp_img/create_style which creates styles with unusable names
creating styles on the form on admin/config/media/breakpoints/create_style creates more usable names
best adding a underscore at the end of the base name
Comment #4
jelle_sFixed in latest dev (reusing the form from the Breakpoints module)
Comment #5
jelle_sUnassigning