When clicking the Apply button to search for specific videos, it switches to the Upload tab when the results are loaded. I have to manually click the YouTube tab to see the results.
This might confuse users until they realize how it works. However, it should really stay on the YouTube tab.
When submitting the form through one of the buttons: Apply, Next or Previous the page is refreshed without adding the correct fragment. In this case '#media-tab-youtube'.
See #5 for steps to reproduce.
XREF : Part of this issue depends on media #1802026-6: Ajax callbacks in the media overlay cause the active tab to switch tab unexpectedly
Comment | File | Size | Author |
---|---|---|---|
#11 | before.jpg | 391.32 KB | gmclelland |
#11 | after.jpg | 518.73 KB | gmclelland |
#7 | youtube-next-previous-links-1551376-7.patch | 706 bytes | aaron |
Comments
Comment #1
gmclelland CreditAttribution: gmclelland commentedSimilar thing happens in this issue #1593624: YouTube tab: Next Link doesn't work correctly
Comment #2
RobW CreditAttribution: RobW commentedComment #3
dddave CreditAttribution: dddave commentedClosed #1815132: When loading search results media youtube tab switches to upload as a dupe.
Comment #4
mrfelton CreditAttribution: mrfelton commentedI think this is the same as the Media bug I reported #1802026: Ajax callbacks in the media overlay cause the active tab to switch tab unexpectedly
Comment #5
clemens.tolboomThe cause for this issue and #1593624: YouTube tab: Next Link doesn't work correctly is that the media browser or more specific jQuery UI tab does not get the fragment #media-youtube
Testing this with
http://example.com/media/browser?render=media-popup#media-tab-youtube
selects the correct tab. Make sure you refresh the page when editing the fragment into ie.
media/browser?render=media-popup#media-tab-media_internet
So we need to fix this either through media_youtube or media module to add the fragment. jQuery UI Tabs then uses this.
Comment #6
clemens.tolboomComment #7
aaron CreditAttribution: aaron commentedThis patch will fix the issue with the next/previous links. I still need to address the search button itself.
Comment #8
clemens.tolboom@aaron nice.
For the Search Button we probably need media to fix for that iframe fragment in line with
(http://stackoverflow.com/questions/4299435/remember-which-tab-was-active...)
But this is more a solution for the container aka media module
using the setter (and getter?) like (http://api.jqueryui.com/tabs/#option-active)
(hope this helps)
Comment #9
gmclelland CreditAttribution: gmclelland commentedThanks @aaron and @clemens.tolboom. The patch in #7 works, but it also makes the page jump to the top of the media browser dialog each time you click on next or previous links. I'm not sure if there is a way around that. It's still an improvement.
Comment #10
clemens.tolboomI added my media related comment from #8 to #1802026-5: Ajax callbacks in the media overlay cause the active tab to switch tab unexpectedly.
@gmclelland what do you mean by 'jumps to the top'. Is the dialog not high enough? Please search for a bug report in media or media_youtube or create a new one :-)
Comment #11
gmclelland CreditAttribution: gmclelland commented@clemens.tolboom - See the screenshots for an example of how the dialog jumps.
The first screenshot shows before I click "next"
The second screenshot shows the after. Notice how the media browser is at the top of the page?
Hope that helps illustrate what I'm talking about.
Comment #11.0
gmclelland CreditAttribution: gmclelland commentedAdded comment #5 partly
Comment #12
clemens.tolboomI think I fixed the Apply problem in media: #1802026-6: Ajax callbacks in the media overlay cause the active tab to switch tab unexpectedly
So this is now only about Next & Previous and only needs a review.
@gmclelland please check the issue queue(s) for your bug which is probably media.module related.
Comment #13
clemens.tolboomPatch #7 works like a charm.
The apply has a follow up in #1802026-6: Ajax callbacks in the media overlay cause the active tab to switch tab unexpectedly
Comment #14
torotil CreditAttribution: torotil commentedI can still reproduce the tab-switching with the 2.0-alpha1 media release and the patch mentioned in this issue: Searching for youtube-videos jumps to the upload tab …
Comment #15
jlk4p CreditAttribution: jlk4p commentedI still reproduce this next/previous problem with Media 2.0-alpha2/Youtube 2.0-rc3 releases. The patch applied for Media 7.x-2.0 in comment 31 for #1802026: Ajax callbacks in the media overlay cause the active tab to switch tab unexpectedly does eliminate the tab from changing if the apply button is clicked.
Comment #15.0
jlk4p CreditAttribution: jlk4p commentedUpdated issue summary.