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.
When the media module is enabled I get weird borders and spacing in rubik. See screenshot attached.
Comment | File | Size | Author |
---|---|---|---|
#13 | Create Article Sportswriters.com 2.0.png | 26.61 KB | rerooting |
#12 | Capture d’écran 2012-09-07 à 12.54.53.png | 25.96 KB | camdarley |
#6 | 1623510-6-rubik-widget-styling.patch | 5.02 KB | Jackinloadup |
#5 | search-button-style.png | 11.81 KB | CoffeyMachine |
#5 | missing-top-border.png | 13.85 KB | CoffeyMachine |
Comments
Comment #1
codi CreditAttribution: codi commentedCleaned up the media widget for image and videos. Added some nested form-item selectors to remove excess borders and added a border to the form-wrapper class.
Comment #2
pschuelke CreditAttribution: pschuelke commentedI re-rolled codi's patch #1 with dave's patch (issue 1418412 #2) and added additional styles
Comment #3
pschuelke CreditAttribution: pschuelke commentedone more little thing added. this replaces #2.
Comment #4
rerooting CreditAttribution: rerooting commentedConfirmed that this fixes everything, including issues I had with the media browser in 2.x . Thanks!
a.button is also really handy when using things like panels + rules_link for fancy interfaces, so supporting a snazzy a.button style will be very useful!
Comment #5
CoffeyMachine CreditAttribution: CoffeyMachine commentedThis is looking good, but there are still a few minor issues that I'd like to be fixed before committing this patch. See the attached screenshots.
Thank you for all your work, once we can tighten these things up I will be happy to commit the patch.
Comment #6
Jackinloadup CreditAttribution: Jackinloadup commentedThis should fix the issues mentioned in #5.
Also used rubik icons for the media buttons.
Comment #7
Jackinloadup CreditAttribution: Jackinloadup commentedComment #8
CoffeyMachine CreditAttribution: CoffeyMachine commentedHi, everyone. Thank you for all your work on this.
I took Jackinloadup's patch and tweaked several very minor things, some padding, font styles, and the launcher button style, and I'm very happy with how it looks.
I've committed this to the 7.x-4.x branch.
Comment #9
rerooting CreditAttribution: rerooting commentedYeah it looks great! My mistake on confirming too early, I overlooked some details. Also works with epsacrop and other modules that add new buttons and components to the media field widget.
Comment #10
k4v CreditAttribution: k4v commentedworks great for me =)
Comment #11
rerooting CreditAttribution: rerooting commentedOne more thing!
In file_entity 2.x, the background image for the modal media 'edit file' form does not repeat horizontally for whatever reason and looks pretty wonky. I'm just recanting this from memory, should be able to fix it with a very amount of css.
Comment #12
camdarley CreditAttribution: camdarley commentedIs the .form-wrapper border useful? It create ugly border on Rules elements.
Comment #13
rerooting CreditAttribution: rerooting commented@camdarley Yeah I saw that. Possibly we would need to be more selective of applying it to form-wrapper, I find that it's useful for what it was applied for.
I'll also see if I can't get around to fixing the edit file modal in #11 for file_entity, has anyone gave that a shot? I know that using media-2.x/file-entity-2.x in this way is new territory for some, and I'd be glad to explain how to reproduce it.
Heres a screenshot of what i'm talkin about:
Comment #15
prinds CreditAttribution: prinds commentedI have opened a seperate issue about the border problems mentioned in #12 here #2155277: Border on .form-wrapper