I know it's late to bring this up, since colorbox support is firmly embedded in this module. But I don't understand why we support colorbox instead of overlay module.
A lightbox (colorbox) is typically used for display of static content, like images or even video.
A modal frame (overlay) is for elements that have more interactivity; ie items that might have a form or links that you want to follow within the modal frame.
It seems to me that nodes more often fall into the second category than the first. Switching to support overlay would also eliminate all those pesky Ux problems around what happens when someone clicks the "edit" tab, or tries to comment, or follows a link etc.
Comment | File | Size | Author |
---|---|---|---|
#13 | fullcalendar_overlayoption_1.patch | 4.92 KB | milos.kroulik |
#10 | fullcalendar_overlayoption.patch | 5.1 KB | jmary |
#4 | 1598712-fullcalendar-overlay.patch | 3.29 KB | ohthehugemanatee |
#2 | 1598712-fullcalendar-overlay.patch | 3.32 KB | ohthehugemanatee |
Comments
Comment #1
tim.plunkettThe core overlay module is only used for admin pages, or optionally, content editing pages. It's never used to display content.
However, I'm not against this idea altogether.
Comment #2
ohthehugemanatee CreditAttribution: ohthehugemanatee commentedIt's pretty straightforward to use it for displaying content... http://drupal.org/project/overlay_paths makes it very easy to use it throughout your site.
Attached a patch that adds overlay support. I basically just copied what you had for colorbox to learn how it works. Probably more thought needs to go into the Views plugin interface, because I don't know what would happen if you select both Overlay and Colorbox. It would get interesting.
Anyway, this is enough as a start.
For people who have trouble getting their Admin theme to apply to nodes, see http://drupal.org/project/admin_theme .
There's also a core bug that can get in the way if you're using the overlay with path aliases; there's a D8 patch at #1240704: hook_admin_paths() doesn't work with path aliases that applies cleanly to D7.
Comment #3
tim.plunkett...
Comment #4
ohthehugemanatee CreditAttribution: ohthehugemanatee commenteddammit - just found that myself. lol - I was debugging it and forgot to take that out. Well, now the world knows that I have a puerile sense of humor.
Sorry about that - rerolled patch attached.
(boy is my face red!)
Comment #5
tim.plunkettI guess its not too bad, my coworker uses "your mom".
Comment #6
muschpusch CreditAttribution: muschpusch commentedSince we have ctools as a dependency why not using ctools overlay with some nice theming options? A lot of people disable the overlay module...
Comment #7
geek-merlin+1 for #6: use ctools overlay.
see also
http://drupal.org/project/automodal
http://drupal.org/project/ctools_automodal
Comment #8
milos.kroulik CreditAttribution: milos.kroulik commentedIt works fine for me, I just had to also enable paths in Overlay Paths module.
I think, that it can be commited.
Comment #9
jmary CreditAttribution: jmary commentedThat' s a good idea, because clicking on event is not always going on the node view page. On my project, it goes to the node edit.
Comment #10
jmary CreditAttribution: jmary commentedThe command
patch -p1 < 1598712-fullcalendar-overlay.patch
failed because of bad formatted patch.I have recreated the patch with command :
diff -rupN fullcalendar.old fullcalendar.new > fullcalendar_overlayoption.patch
Now it applies properly.
Comment #11
jmary CreditAttribution: jmary commentedComment #12
milos.kroulik CreditAttribution: milos.kroulik commentedThe patch from #10 also seems to work fine.
EDIT: There is a problem, though, that generates dummy
sites/all/modules...
directory structure inside module's directory. I will add my modified patch, that corrects this issue.Comment #13
milos.kroulik CreditAttribution: milos.kroulik commented