mirrors/TiddlyWiki5
1
0
Fork 0
mirror of https://github.com/Jermolene/TiddlyWiki5 synced 2024-11-30 05:19:57 +00:00
Code Issues Releases Wiki Activity

Make navigation in new windows work for storyviews (#5759)

Browse Source
This commit is contained in:
Simon Huber 2021-06-06 11:47:19 +02:00 committed by GitHub
parent 2b911ac11f
commit afa4ea3d03
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
3 changed files with 9 additions and 9 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
core/modules/storyviews/classic.js
View File

@ -27,7 +27,7 @@ ClassicStoryView.prototype.navigateTo = function(historyInfo) {
var listItemWidget = this.listWidget.children[listElementIndex], var listItemWidget = this.listWidget.children[listElementIndex],
targetElement = listItemWidget.findFirstDomNode(); targetElement = listItemWidget.findFirstDomNode();
// Abandon if the list entry isn't a DOM element (it might be a text node) // Abandon if the list entry isn't a DOM element (it might be a text node)
if(!(targetElement instanceof Element)) { if(targetElement === undefined || targetElement.nodeType === Node.TEXT_NODE) {
return; return;
} }
if(duration) { if(duration) {
@ -43,7 +43,7 @@ ClassicStoryView.prototype.insert = function(widget) {
if(duration) { if(duration) {
var targetElement = widget.findFirstDomNode(); var targetElement = widget.findFirstDomNode();
// Abandon if the list entry isn't a DOM element (it might be a text node) // Abandon if the list entry isn't a DOM element (it might be a text node)
if(targetElement.nodeType === Node.TEXT_NODE) { if(targetElement === undefined || targetElement.nodeType === Node.TEXT_NODE) {
return; return;
} }
// Get the current height of the tiddler // Get the current height of the tiddler
@ -83,7 +83,7 @@ ClassicStoryView.prototype.remove = function(widget) {
widget.removeChildDomNodes(); widget.removeChildDomNodes();
}; };
// Abandon if the list entry isn't a DOM element (it might be a text node) // Abandon if the list entry isn't a DOM element (it might be a text node)
if(targetElement.nodeType === Node.TEXT_NODE) { if(targetElement === undefined || targetElement.nodeType === Node.TEXT_NODE) {
removeElement(); removeElement();
return; return;
} }

6
core/modules/storyviews/pop.js
View File

@ -24,7 +24,7 @@ PopStoryView.prototype.navigateTo = function(historyInfo) {
var listItemWidget = this.listWidget.children[listElementIndex], var listItemWidget = this.listWidget.children[listElementIndex],
targetElement = listItemWidget.findFirstDomNode(); targetElement = listItemWidget.findFirstDomNode();
// Abandon if the list entry isn't a DOM element (it might be a text node) // Abandon if the list entry isn't a DOM element (it might be a text node)
if(!(targetElement instanceof Element)) { if(targetElement === undefined || targetElement.nodeType === Node.TEXT_NODE) {
return; return;
} }
// Scroll the node into view // Scroll the node into view
@ -35,7 +35,7 @@ PopStoryView.prototype.insert = function(widget) {
var targetElement = widget.findFirstDomNode(), var targetElement = widget.findFirstDomNode(),
duration = $tw.utils.getAnimationDuration(); duration = $tw.utils.getAnimationDuration();
// Abandon if the list entry isn't a DOM element (it might be a text node) // Abandon if the list entry isn't a DOM element (it might be a text node)
if(targetElement.nodeType === Node.TEXT_NODE) { if(targetElement === undefined || targetElement.nodeType === Node.TEXT_NODE) {
return; return;
} }
// Reset once the transition is over // Reset once the transition is over
@ -77,7 +77,7 @@ PopStoryView.prototype.remove = function(widget) {
} }
}; };
// Abandon if the list entry isn't a DOM element (it might be a text node) // Abandon if the list entry isn't a DOM element (it might be a text node)
if(targetElement.nodeType === Node.TEXT_NODE) { if(targetElement === undefined || targetElement.nodeType === Node.TEXT_NODE) {
removeElement(); removeElement();
return; return;
} }

6
core/modules/storyviews/zoomin.js
View File

@ -48,7 +48,7 @@ ZoominListView.prototype.navigateTo = function(historyInfo) {
var listItemWidget = this.listWidget.children[listElementIndex], var listItemWidget = this.listWidget.children[listElementIndex],
targetElement = listItemWidget.findFirstDomNode(); targetElement = listItemWidget.findFirstDomNode();
// Abandon if the list entry isn't a DOM element (it might be a text node) // Abandon if the list entry isn't a DOM element (it might be a text node)
if(!(targetElement instanceof Element)) { if(targetElement === undefined || targetElement.nodeType === Node.TEXT_NODE) {
return; return;
} }
// Make the new tiddler be position absolute and visible so that we can measure it // Make the new tiddler be position absolute and visible so that we can measure it
@ -130,7 +130,7 @@ function findTitleDomNode(widget,targetClass) {
ZoominListView.prototype.insert = function(widget) { ZoominListView.prototype.insert = function(widget) {
var targetElement = widget.findFirstDomNode(); var targetElement = widget.findFirstDomNode();
// Abandon if the list entry isn't a DOM element (it might be a text node) // Abandon if the list entry isn't a DOM element (it might be a text node)
if(targetElement.nodeType === Node.TEXT_NODE) { if(targetElement === undefined || targetElement.nodeType === Node.TEXT_NODE) {
return; return;
} }
// Make the newly inserted node position absolute and hidden // Make the newly inserted node position absolute and hidden
@ -147,7 +147,7 @@ ZoominListView.prototype.remove = function(widget) {
widget.removeChildDomNodes(); widget.removeChildDomNodes();
}; };
// Abandon if the list entry isn't a DOM element (it might be a text node) // Abandon if the list entry isn't a DOM element (it might be a text node)
if(targetElement.nodeType === Node.TEXT_NODE) { if(targetElement === undefined || targetElement.nodeType === Node.TEXT_NODE) {
removeElement(); removeElement();
return; return;
} }