Closed (outdated)
Project:
Quick Tabs
Version:
7.x-3.4
Component:
Code
Priority:
Normal
Category:
Feature request
Assigned:
Unassigned
Reporter:
Created:
26 Nov 2010 at 22:45 UTC
Updated:
11 Dec 2024 at 17:34 UTC
Jump to comment: Most recent
Comments
Comment #1
WoozyDuck commentedSame request here
Comment #2
fourmi4x commentedWould love to see that too...
A good start could be : where are the URLs constructed (which file/line) ?
Comment #3
fourmi4x commentedI worked out a temporary workaround by modifying some lines in the quicktabs.module file.
---------------
The first bit replaces the second "quicktabs" word by the name of the tab (converted into an url format) :
Replace the 2 following lines by these :
'id' => strtolower(str_replace(array(' ', ' '), '-', preg_replace('/[^a-zA-Z0-9 s]/', '', trim($tab['title'])))) . '-' . $quicktabs['qtid'],'fragment' => strtolower(str_replace(array(' ', ' '), '-', preg_replace('/[^a-zA-Z0-9 s]/', '', trim($tab['title'])))) . '-' . $qtid,---------------
The second bit replaces the first "quicktabs" word with the node title :
Replace the 4 following lines by these :
unset($query[$tabname . $qtid]);$query[$tabname . $qtid] = $tabkey;$active_tab = isset($_GET[$tabname . $quicktabs['qtid']]) ? $_GET[$tabname . $quicktabs['qtid']] : $active_tab;$active_tab = isset($_GET[$tabname . $quicktabs['qtid']]) ? $_GET[$tabname . $quicktabs['qtid']] : $active_tab;After that, add these 3 lines after line 357 to get the node title into the tab url :
Line 357 is // Need to construct the correct query for the tab links.
and then add :
and then it continues with
$query = $_GET; which should be now line 361 !
---------------
Hope that helps. Beware it is not fully tested, but it works for me ! (version 6.x-2.0-rc)
Sorry I don't know how to make a patch, but this shouldn't really go in a patch anyway, it is just a temporary fix...
What would be awesome would be tu be able to use tokens directly into these URLs !!
Comment #4
cbrody commentedSubscribing
Comment #5
urbanbricks commentedSubscribing
Comment #6
ParisLiakos commentedSubscribe and agree..
?quicktabs_1=1#quicktabs-1 is VERY ugly
Comment #7
ParisLiakos commentedComment #8
doublejosh commentedEdit:
Path aliasComment #9
doublejosh commentedJust did redirects so that we can give out nice URLs to people, but sure would rock to have them as a config feature.
Perhaps there a path to the Path module that allows fragments?
Comment #10
Rameez commentedSubscribing
really imp issue. However i'm interested to know if we can completely remove links on tab? wat if they behave like buttons?
Comment #11
aalireza commentedSubscribing
Comment #12
goldlilys commentedThis feature would also be useful for D7 version since the current url is UGLY. If not a custom url path, can it be the title of the tab itself instead of #quicktabs-page=1 ... have #quicktabs-page=tab-title OR #page=tab-title OR just #tab-title
Comment #13
Ndesign commentedSubscribing
Comment #14
Exploratus commentedSubscribe.
Comment #15
smustgrave commentedWith D7 EOL approaching in a month I'm starting to triage the D7 side of quicktabs queue. This doesn't appear to have any code so believe this may not make it, sorry! Thanks though!
If still an issue or needed for 4.0.x (latest branch) feel free to reopen