Feature/#19 notifications tab title #31
|
|
@ -6,7 +6,7 @@ export function init() {
|
|||
updateNotificationsInTitle();
|
||||
}, ".*");
|
||||
|
||||
core.runAfterPathnameChange(() => {
|
||||
core.runAfterLocationChange(() => {
|
||||
updateNotificationsInTitle();
|
||||
Do you remember how exactly you produced that bug with the number written in the title multiple times? I haven´t seen that yet. But something else I noticed was the fact, the title seems to be updated by aniwatch after the blue loading bar on the top disappears. This should be investigated as a new opportunity to decide when we update the title and also would fix the current issue, that we just guess a random number to wait until location change before we update the title, which doesn´t always works as the following gif shows: Do you remember how exactly you produced that bug with the number written in the title multiple times? I haven´t seen that yet.
But something else I noticed was the fact, the title seems to be updated by aniwatch after the blue loading bar on the top disappears. This should be investigated as a new opportunity to decide when we update the title and also would fix the current issue, that we just guess a random number to wait until location change before we update the title, which doesn´t always works as the following gif shows:
Yeah thats what I meant. Looks like we should not only wait for the popState but also for onload once their preloader is fixed, or wait for the loading bar to disappear(?) Yeah thats what I meant. Looks like we should not only wait for the popState but also for onload once their preloader is fixed, or wait for the loading bar to disappear(?)
I introduced another observer, which will execute after the blue loading bar in the top disappears. This error seems not to appear any more with that specific change. I introduced another observer, which will execute after the blue loading bar in the top disappears. This error seems not to appear any more with that specific change.
- [x] There still is the problem, that some pages have a different loading method where the menu isn´t available directly in the DOM.
I was able to fix this. We started our scripts after the preloader disappeared. We now also wait for the I was able to fix this. We started our scripts after the preloader disappeared. We now also wait for the `document.readyState` to be `"complete"` so we can ensure *everything* is loaded correctly, as the preloader disappears to early on some pages.
|
||||
}, ".*");
|
||||
}
|
||||
|
|
|
|||
|
|
@ -3,7 +3,7 @@ import * as helper from './helpers';
|
|||
/* SCRIPT LOGICS */
|
||||
let __scripts = [];
|
||||
let __afterLoadScripts = [];
|
||||
let __afterPathnameChangeScripts = [];
|
||||
let __afterLocationChangeScripts = [];
|
||||
|
||||
export function initCore() {
|
||||
let observer = new MutationObserver(mutations => {
|
||||
|
|
@ -20,16 +20,13 @@ export function initCore() {
|
|||
attributes: true
|
||||
});
|
||||
|
||||
// patchBrowser();
|
||||
// window.addEventListener('locationchange', (event) => handleLocationChanged(event));
|
||||
|
||||
runAfterLoad(() => {
|
||||
let loadingBar = document.getElementById('enable-ani-cm');
|
||||
let loadingBarObserver = new MutationObserver(mutations => {
|
||||
mutations.forEach(mutation => {
|
||||
// enable-ani-cm node changes from display:none to display:block after loading
|
||||
if (mutation.oldValue.includes('display: none')) {
|
||||
__afterPathnameChangeScripts.forEach(script => {
|
||||
__afterLocationChangeScripts.forEach(script => {
|
||||
if (window.location.pathname.match(script.pattern)) {
|
||||
script.function();
|
||||
}
|
||||
|
|
@ -100,45 +97,8 @@ function awaitPageLoaded() {
|
|||
}
|
||||
|
||||
/* PATHNAME LOGIC */
|
||||
export function runAfterPathnameChange(func, pattern = '.*') {
|
||||
__afterPathnameChangeScripts.push({ "function": func, "pattern": pattern });
|
||||
}
|
||||
|
||||
function handleLocationChanged(event) {
|
||||
__afterPathnameChangeScripts.forEach(script => {
|
||||
if (window.location.pathname.match(script.pattern)) {
|
||||
script.function();
|
||||
}
|
||||
});
|
||||
}
|
||||
|
||||
function patchBrowser() {
|
||||
// patches several browser functions to dispatch a "locationchange" event
|
||||
// as an extension is not allowed to override these functions we have to inject this as a script tag into the head
|
||||
let scriptContent = `history.pushState = (func => function pushState() {
|
||||
let result = func.apply(this, arguments);
|
||||
window.dispatchEvent(new Event('pushstate'));
|
||||
window.dispatchEvent(new Event('locationchange'));
|
||||
|
||||
return result;
|
||||
})(history.pushState);
|
||||
|
||||
history.replaceState = (func => function replaceState() {
|
||||
let result = func.apply(this, arguments);
|
||||
window.dispatchEvent(new Event('replacestate'));
|
||||
window.dispatchEvent(new Event('locationchange'));
|
||||
return result;
|
||||
})(history.replaceState);
|
||||
|
||||
window.addEventListener('popstate', () => {
|
||||
window.dispatchEvent(new Event('locationchange'))
|
||||
});`
|
||||
|
||||
let head = document.getElementsByTagName("head")[0];
|
||||
let newScript = document.createElement('script');
|
||||
newScript.type = 'text/javascript';
|
||||
newScript.innerHTML = scriptContent;
|
||||
head.appendChild(newScript);
|
||||
export function runAfterLocationChange(func, pattern = '.*') {
|
||||
__afterLocationChangeScripts.push({ "function": func, "pattern": pattern });
|
||||
}
|
||||
|
||||
/* LOGIN LOGIC */
|
||||
|
|
|
|||
Getting the NotificationCount on every pushState event increases the chance of it failing. With this solution it fails for every item of the Community and Anime dropdownmenus. With the old solution it failed when inserting the Count into the title for the stats, jobs, seasonal, airing, random, profile, watchlist and notifications. 10 vs 8 pathnames where it doesn't work.
Furthermore with this solution we run into problems on the profile pathname:
We are adding the NotificationCount to the old count whenever we switch from one tab under profile to another; this did not happen with the old solution.
This seems to be related to #42 tho, if you enable Low-end mobile and check the pathnames after visting /home first, you'll see the notification count appear for a while and then disappear once the site is fully shown. With the new solution its definitely because of #42; we are trying to access the menu while the page isn't fully loaded.