if ($configuration->settings->mma_node_edit_item_show)

should also encapsule the table header

i added another line instead... didn't find any issues about this

CommentFileSizeAuthor
#1 media_mover_api.patch711 bytesdiscipolo
medialist.patch562 bytesdiscipolo

Comments

discipolo’s picture

discipolo commented
Title: when not set to show files, hide configuration fieldset also » mediamover files (fieldset) is not hidden
StatusFileSize
new media_mover_api.patch711 bytes

heres a newer patch against media_mover_api.module 6.x-1.0-beta7 really looking forward to 6.x-2 !

bryanb229’s picture

bryanb229 commented

Thanks discipolo.

Looking forward to 6.x-2, also!

arthurf’s picture

arthurf commented
Status: Active » Needs review

Thanks for the patch. Committed to 6.1.x, was already in place on 6.2.x

bcmiller0’s picture

bcmiller0 commented

didn't see this in 6.x-1.0-beta9? guess still needs review?

arthurf’s picture

arthurf commented

Beta 9 is actually beta 7. I'm not sure why but there was a bunch of stuff wrong when I cut a tag from my dev branch. The fix is in the dev branch, so if I can figure out what went wrong it will be in the next release

discipolo’s picture

discipolo commented
Status: Needs review » Closed (fixed)

closing