Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
This is probably my fault due to funky markup and css from HEAD, but check the eldir vs non eldir pics.
this is a critical issue, as it's one of the things i specifically need to cover in the install screencast.
Comment | File | Size | Author |
---|---|---|---|
#6 | Queue admin form in Firefox | 105.35 KB | omega8cc |
#6 | Queue admin form in Safari | 100.85 KB | omega8cc |
#6 | Queue admin form in Chrome | 99.54 KB | omega8cc |
#4 | eldir-hardcore-fix.patch | 1.17 KB | omega8cc |
#3 | 0001-460974-improving-queue-admin-form.patch | 1.18 KB | Pls |
Comments
Comment #1
yhahn CreditAttribution: yhahn commentedhttp://drupal.org/cvs?commit=210674
I've put in a stopgap fix for now but it's not as good as it could be. We should revisit this and the larger general issue here -- limited real estate for forms that could use a bit more room.
One option would be to switch to a fluid layout, but Eldir is currently at 960 which is what a good portion of users will max out at anyway even with fluid width. Other options worth exploring might be to look at alternative layouts that don't use 2 columns/sidebars for navigation/status blocks.
Comment #2
anarcat CreditAttribution: anarcat commentedI'm not sure the larger issue belongs here, but at any rate, the original bug is 'fixed enough' so that we can release hence I'm downgrading this to 'normal'.
Comment #3
Pls CreditAttribution: Pls commentedI checked original bug report and it seems fixed. Now added a patch to improve queue admin form. Feedback is appreciated. Screens attached - before and after. Thanks.
Comment #4
omega8cc CreditAttribution: omega8cc commentedThis Eldir patch works fine for Chrome, but not for Safari, where the "sites every" is displayed *after* (on the right) all the form fields (for Cron queue) instead of above them. Not sure how to fix this in CSS since the text is not wrapped in any div or span, so I'm attaching a patch to show where is the problem (however it is not a solution, probably).
Comment #5
Steven Jones CreditAttribution: Steven Jones commentedThis is still an issue in the latest version of Eldir, and should be an easy fix I reckon.
Comment #6
omega8cc CreditAttribution: omega8cc commentedI think we have it fixed in our Eldir fork: http://drupalcode.org/sandbox/omega8cc/1074912.git/blob/HEAD:/themes/eld...
This forked version includes fixes even for some popular contrib modules, as you can see.
I'm attaching the screenshots from Firefox, Safari and Chrome.
Comment #7
omega8cc CreditAttribution: omega8cc commentedSo, I guess we should submit a proper patch for review.
Comment #8
Steven Jones CreditAttribution: Steven Jones commentedI've re-worked the CSS on this form so that it applies generically and doesn't need fixing up for each queue in 6.x-2.x.