-
Notifications
You must be signed in to change notification settings - Fork 12
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
send2cgeo button not visible on cache page #62
Comments
this is an other issue than cgeo/cgeo#8061 |
I can confirm. Seems the script is broken as there are changes to the website?! |
I am using Tampermonkey in Firefox 64.0.1 with the latest @Lineflyer It seems like you just forgot to increase the version numbers for the changed user scripts (#60 and #61 being the recent PRs) and merge the updated versions into the |
@FriedrichFroebel I pushed tthe mentioned PRs to master first to give them a test before rolling out the change by merge to |
I don't have any javascript expertise. Line 296 in fb23515
|
Then we may try to include jQuery for it - this should fix the problem (see https://stackoverflow.com/questions/24753908/trying-to-load-jquery-into-tampermonkey-script). The current jQuery version is 3.4.1. (In my case jQuery seems to be loaded by the GC Little Helper II script, so this does not raise an error.) |
With adding jQuery it works on my Firefox. |
indeed, added // @require http://code.jquery.com/jquery-3.4.1.min.js to send2cgeo script (before all //@include lines) resolved issue in both Firefox (72.0.1 (64-bit)) and Chrome (79.0.3945.117 (Official Build) (64-bit)) |
Thanx, // ==UserScript== // Inserts javascript that will be called by the s2cgeo button. The closure /* global s2geomulti */ var s = document.createElement('script'); s.type = 'text/javascript'; window.s2geo = function(code) { function s2geomultiProcessLine(toAddIframe, skipFound, requestedCnt, alreadySent) { window.s2geomulti = function(requestedCnt,skipFound,moreClicked) { $("#searchResultsTable").before( <a href="#" onclick="window.s2geomulti(50); return false;">Send2cgeo: 50 " "+ "<a href="#" onclick="window.s2geomulti(100); return false;">100 " + "<a href="#" onclick="window.s2geomulti(200); return false;">200 " + "<a href="#" onclick="window.s2geomulti(500); return false;">500 " + "<span class="label">Skip found caches ); // check for premium membership (parts of the page content are different)
} // this adds a column with send2cgeo button in search results table
} // waits for new elements (by ajax calls) injected into the DOM and calls a certain
} window.waitForKeyElements(".cache-details", addSend2cgeoColumn, false); // Defines the elements to insert into the page ////////////////////////////// var boxStyle = 'display:none; background:#1D1D1D; z-index:1000; left:50%;' $('body').append(' ' ');
+ ' Please wait… '+ '<iframe style="'+iframeStyle+'"></iframe>' + ' // Append to send2cgeo links/buttons ///////////////////////////////////////// if(map !== null) {
} else if(document.getElementById('searchResultsTable') != null){
}
} else if(document.getElementById('ctl00_ContentBody_CoordInfoLinkControl1_uxCoordInfoCode') != null){
} else { } // Inject Script. Can't use jQuery yet, because the page is not |
With the suggested addition it is not possible to open a geocache-page from the map |
The implementation for the map is completely broken with the GC new map. |
My point is that with the remedy of av72 the geocaches are not clickable(and therefor will not open in a new tab) any more on the geocache map in chrome. Without the remedy (or not running the script) on the mapping page the caches do open in a new page on clicking |
I presume there are two issues here:
I would suggest to keep this #62 strictly to the button on the detailed cache page; it then can be closed. Directly opening a new detailed cache page when clicking on the map can then be resolved under #63, I suspect this will have significantly more impact. |
@wizard4u |
sendtogeo not posssible on geocache pages no tab click visible
in chrome and firefox
The text was updated successfully, but these errors were encountered: