Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Added tooltip option to Tabs macro #2147

Merged
merged 2 commits into from
Dec 24, 2015
Merged

Added tooltip option to Tabs macro #2147

merged 2 commits into from
Dec 24, 2015

Conversation

matabele
Copy link
Contributor

Tooltips should be entered in the 'tooltip' field of each tab tiddler

Tooltips should be entered in the 'tooltip' field of each tab tiddler
@matabele
Copy link
Contributor Author

In this version -- 'currentTiddler' is set to the value of 'currentTab' only around the $button widget. Hopefully this will not interfere with anything else (appears to be no problems.)

@matabele
Copy link
Contributor Author

This seems to break the "default" tab -- but I can't see why. Any ideas?

Specifically -- the default is broken only for subsequent calls of the tabs macro (as for the examples) -- the first call in any tiddler functions correctly.

@matabele
Copy link
Contributor Author

OK -- after cleaning all of the state tiddlers, now functions correctly -- seems to have been caused by 'dirty' leftover state tiddlers.

What do you think of this idea?
@Jermolene
Copy link
Member

Thanks @matabele

Jermolene added a commit that referenced this pull request Dec 24, 2015
Added tooltip option to Tabs macro
@Jermolene Jermolene merged commit 7ef60de into TiddlyWiki:master Dec 24, 2015
Jermolene added a commit that referenced this pull request Jan 7, 2016
The problem is that this change scrambles the ordering of tabs that use
programmatic ordering, for example here:

https://github.com/Jermolene/TiddlyWiki5/blob/master/editions/tw5.com/ti
ddlers/releasenotes/TiddlyWiki_Releases.tid#L13
@matabele matabele deleted the patch-1 branch January 7, 2016 19:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants