"Plaats in discussietopic" support topic

[ToolkiT]

Modbreak:

@Devs: Ik ben even zo vrij om het bugforum te gebruiken voor support voor de "plaats in discussietopic" feature in het GGPT, middels html in de topicwaarschuwing .

-.oisyn

@modjes de 'post in discussie topic' knop is weer weg

[ Voor 63% gewijzigd door .oisyn op 26-07-2024 12:34 ]

[Touchdomex]

ToolkiT schreef op dinsdag 23 juli 2024 @ 10:12:
[Afbeelding]

Deze quote is mede mogelijk gemaakt door de plaats in discussietopic-knop.

ToolkiT schreef op vrijdag 26 juli 2024 @ 10:51:
@modjes de 'post in discussie topic' knop is weer weg

Doet het hier prima en staat er ook tussen. Denk dat het probleem zich aan de andere kant van het kanaal bevind

[ Voor 27% gewijzigd door Touchdomex op 26-07-2024 10:57 ]

[ToolkiT]

Touchdomex schreef op vrijdag 26 juli 2024 @ 10:56:
[...]


Deze quote is mede mogelijk gemaakt door de plaats in discussietopic-knop.


[...]


Doet het hier prima en staat er ook tussen. Denk dat het probleem zich aan de andere kant van het kanaal bevind

Ben ik nou gek of is ie gewoon weg?

Mischien een browser specifiek issue?
ik gebruik gener8

[mrc4nl]

ToolkiT schreef op vrijdag 26 juli 2024 @ 11:07:
[...]

Ben ik nou gek of is ie gewoon weg?
[Afbeelding]
Mischien een browser specifiek issue?
ik gebruik gener8

same hierzo.

Knop is er gewoon niet

[gambieter]

Ik zie hem ook al een tijdje niet op mobiel. Zal straks eens kijken op desktop.

[Osiris]

Ik heb 'em nog wel hoor.

[ToolkiT]

Osiris schreef op vrijdag 26 juli 2024 @ 11:23:
Ik heb 'em nog wel hoor.

made me look

[Notna]

ToolkiT schreef op vrijdag 26 juli 2024 @ 11:07:
[...]

Ben ik nou gek of is ie gewoon weg?
[Afbeelding]
Mischien een browser specifiek issue?
ik gebruik gener8

Het één sluit het ander niet uit hé

@.oisyn ˆˆˆˆ Jij een idee?

[Touchdomex]

ToolkiT schreef op vrijdag 26 juli 2024 @ 11:07:
[...]

Ben ik nou gek of is ie gewoon weg?
[Afbeelding]
Mischien een browser specifiek issue?
ik gebruik gener8

Makkelijkste test is een browser te pakken die zo kaal mogelijk is en gene rare dingen toepast zoals gener8 dat wel lijkt te doen met ads bijvoorbeeld.

[ToolkiT]

Touchdomex schreef op vrijdag 26 juli 2024 @ 11:50:
[...]


Makkelijkste test is een browser te pakken die zo kaal mogelijk is en gene rare dingen toepast zoals gener8 dat wel lijkt te doen met ads bijvoorbeeld.

gezien de reacties hierboven ben ik geen uitzondering en lijkt het erop dat het een intermittent bug is..
Maar zal eens proberen in een andere browser..

<edit>
in chrome idem dito:


edit2:
ditto in edge:


edit3:
Chrome in incognito mode (om cookies out of the equation te halen):

[ Voor 63% gewijzigd door ToolkiT op 26-07-2024 12:00 ]

[Touchdomex]

ToolkiT schreef op vrijdag 26 juli 2024 @ 11:53:
[...]

gezien de reacties hierboven ben ik geen uitzondering en lijkt het erop dat het een intermittent bug is..
Maar zal eens proberen in een andere browser..

<edit>
in chrome idem dito:
[Afbeelding]

edit2:
ditto in edge:
[Afbeelding]

CustomCSS? de balk bovenin is ook korter dan hier voor zover ik weet.

[ToolkiT]

Touchdomex schreef op vrijdag 26 juli 2024 @ 12:13:
[...]


CustomCSS? de balk bovenin is ook korter dan hier voor zover ik weet.

geloof ik wel.. ooit een keer aangezet en nooit meer wat mee gedaan.. zal eens kijken..

[gambieter]

Ik heb alleen de custom CSS om die belachelijke sidebar uit te schakelen, even uitgezet en maakt mobiel geen verschil.

[ToolkiT]

ik zie hier ook niks tussen wat het zou kunnen veroorzaken..

[.oisyn]

Krijgen jullie errors in de javascript console?

[mrc4nl]

.oisyn schreef op vrijdag 26 juli 2024 @ 12:38:
Krijgen jullie errors in de javascript console?

ik niet.

[.oisyn]

Voor de volledigheid, ik zie 'm gewoon in Chrome op Android

[mrc4nl]

FF 128 , win 10

edit, ook niet met edge 96 op android 13

.oisyn schreef op vrijdag 26 juli 2024 @ 12:47:
Voor de volledigheid, ik zie 'm gewoon in Chrome op Android

even voor de nieuwsgierigheid, waar zou de knop in de broncode moeten staan? wordt de knop pas gegenereerd bij het laden van de pagina, of staat ie al in de html?

[ Voor 96% gewijzigd door mrc4nl op 26-07-2024 12:52 ]

[.oisyn]

@mrc4nl Hij wordt erin gezet door een stukje javascript dat in de topicwaarschuwing staat

[.oisyn]

Dit is de source:

JavaScript:

// Dit script voegt een extra verstuur-knop toe op de plaats-reactie-pagina, omdirect naar het discussietopic te posten!
// Auteur: .oisyn

discussieTopicId = 1494847; // Pas deze aan als het GGPT discussietopic wijzigt!

window.addEventListener("DOMContentLoaded", function() {
var form = document.getElementById("message_form");
if (!form)
    return;

var el = document.querySelector("#message_form input~[type=submit]~[value=Verstuur]");
if (!el)
    return;

var formAction = form.action.substr(0, form.action.lastIndexOf("/") + 1) + discussieTopicId ;
el.insertAdjacentHTML("afterEnd", '<input type="submit" class="ctaButton" value="Plaats in discussietopic" formaction="' + formAction + '">');

});

[.oisyn]

Hmm, misschien vindt die querySelector niets? Wat gebeurt er als je die expression direct in je console uitvoert?

[mrc4nl]

.oisyn schreef op vrijdag 26 juli 2024 @ 12:55:
Hmm, misschien vindt die querySelector niets? Wat gebeurt er als je die expression direct in je console uitvoert?

code:

document.querySelector("#message_form input~[type=submit]~[value=Verstuur]");

-> null

[Dido]

Touchdomex schreef op vrijdag 26 juli 2024 @ 10:56:
Deze quote is mede mogelijk gemaakt door de plaats in discussietopic-knop.
[...]
Doet het hier prima en staat er ook tussen. Denk dat het probleem zich aan de andere kant van het kanaal bevind

Chrome, geen custom css.

[ Voor 14% gewijzigd door Dido op 26-07-2024 13:00 ]

[crisp]

@.oisyn zo te zien zijn er een paar onbedoelde ~'s in de selector gekropen (ws door herparsing, waar een bugje in zat)

[.oisyn]

mrc4nl schreef op vrijdag 26 juli 2024 @ 12:57:
[...]

code:

document.querySelector("#message_form input~[type=submit]~[value=Verstuur]");

-> null

Dan lijkt daar het probleem te liggen, thanks!

[.oisyn]

crisp schreef op vrijdag 26 juli 2024 @ 12:59:
@.oisyn zo te zien zijn er een paar onbedoelde ~'s in de selector gekropen (ws door herparsing, waar een bugje in zat)

Ah ja fijn, thanks voor deze bevestiging! Die snapte ik idd niet maar aangezien ik niet dagelijks css selectors schrijf wist ik het niet zeker

[.oisyn]

@mrc4nl @ToolkiT @Dido kunnen jullie bevestigen dat het nu weer werkt?

[crisp]

.oisyn schreef op vrijdag 26 juli 2024 @ 13:01:
[...]

Ah ja fijn, thanks voor deze bevestiging! Die snapte ik idd niet maar aangezien ik niet dagelijks css selectors schrijf wist ik het niet zeker

Het is alsnog een valide selector, de tilde is een subsequent-sibling combinator

[gambieter]

.oisyn schreef op vrijdag 26 juli 2024 @ 13:03:
@mrc4nl @ToolkiT @Dido kunnen jullie bevestigen dat het nu weer werkt?

@.oisyn ik ben dan wel iemand anders, maar ik kan bevestigen dat ik hem weer zie. Thanks!

Hello button my old friend
I come to press you once again

[Dido]

@.oisyn Affirmative! De knop is terug

* Dido heeft serieus zitten twijfelen aan zijn cognitieve capaciteiten de laatste tijd na een paar keer tekst in het plaatjestopic gezet te hebben
Maar het lag een keertje niet aan mij

[ Voor 70% gewijzigd door Dido op 26-07-2024 13:08 ]

[mrc4nl]

.oisyn schreef op vrijdag 26 juli 2024 @ 13:03:
@mrc4nl @ToolkiT @Dido kunnen jullie bevestigen dat het nu weer werkt?

[gambieter]

mrc4nl schreef op vrijdag 26 juli 2024 @ 13:08:
[...]

[Afbeelding]

Wel een refresh gedaan?

[ToolkiT]

.oisyn schreef op vrijdag 26 juli 2024 @ 13:03:
@mrc4nl @ToolkiT @Dido kunnen jullie bevestigen dat het nu weer werkt?

Thanks for the quick turnaround

[mrc4nl]

gambieter schreef op vrijdag 26 juli 2024 @ 13:12:
[...]

Wel een refresh gedaan?

ja, zelfs ctrl-f5

ook op mn android werkt het hier nog niet

[mrc4nl]

Hee, in de quick react zie ik m niet, maar zodra ik quote, dan wel

[.oisyn]

crisp schreef op vrijdag 26 juli 2024 @ 13:06:
[...]

Het is alsnog een valide selector, de tilde is een subsequent-sibling combinator

Ok maar that begs the question: waarom werkte het voor sommigen dan wel? . Dan moet er dus een input én een element met type=submit aan voorafgegaan zijn.

[.oisyn]

mrc4nl schreef op vrijdag 26 juli 2024 @ 13:19:
Hee, in de quick react zie ik m niet, maar zodra ik quote, dan wel

Dat is by design, je moet wel quoten

[gambieter]

mrc4nl schreef op vrijdag 26 juli 2024 @ 13:19:
Hee, in de quick react zie ik m niet, maar zodra ik quote, dan wel

Ik heb de knop nooit in de QuickReact gezien, alleen in de quote.

[mrc4nl]

dan keek ik al die tijd op de verkeerde plek, whoeps

[.oisyn]

mrc4nl schreef op vrijdag 26 juli 2024 @ 13:22:
dan keek ik al die tijd op de verkeerde plek, whoeps

Toch bedankt voor je bijdrage