INL / GTB-html

GTB zonder Flash
0 stars 0 forks source link

Back-button werkt niet #78

Open jan-niestadt opened 6 years ago

jan-niestadt commented 6 years ago

Dit is niet gebruikersvriendelijk en kan verwarrend zijn.

Beter: als je op Back klikt, krijg je een waarschuwing dat je hiermee de GTB-applicatie verlaat, en dat navigeren binnen de applicatie via de tabbladen werkt.

Best: Back-button werkt zoals gebruikers dat gewend zijn.

pmasereeuw commented 6 years ago

Bekend issue bij applicaties die pagina's aanpassen met scripts. Ik lees dat er wel wat aan te doen moet zijn, ik ga eens kijken, want ik heb nog wat meer plekken waar zo'n oplossing nuttig kan zijn.

Ik zou me willen beperken tot het pushen van de tab-selectie in de browsergeschiedenis, en niet hele query's opnieuw afvuren.

jan-niestadt commented 6 years ago

Weet niet of je dit issue meteen moet oppakken hoor. Misschien voorlopig alleen die waarschuwing implementeren? Een volledige optie, waarbij back/forward, bookmarken van queries, etc. werkt is ws. behoorlijk veel werk.

Aanleiding voor dit issue was een mail van een gebruiker waaruit ik meende op te maken dat die voor elke zoekvraag op zijn bookmark klikte om naar het zoekscherm te gaan. Hij had blijkbaar niet de verschillende tabbladen gezien. Een waarschuwing is daarvoor dus misschien wel handig.

kdepuydt commented 6 years ago

Even pro memorie laten staan. Als we meer meldingen krijgen hiervan, kunnen we er nog altijd wat aan doen.

jan-niestadt commented 6 years ago

Pieter zou al even kijken of hij snel een waarschuwing kon toevoegen als je de pagina verlaat. (hadden we volgens mij ook al in de oude GTB)

pmasereeuw commented 6 years ago

Ik heb een poging gedaan die het leed een beetje verzacht. Ik moet (ooit, misschien) nog uitzoeken hoe ik echt de history bijhoud door de id's van de tabbladen in de history te plaatsen. Ik vermoed dat een deel van de kliks op de tabbladen door bootstrap wordt afgehandeld, dus zal het misschien lastig zijn om dit te centraliseren. Maar ik heb dit nog niet uitgezocht, dus wie weet valt het wel mee.

jan-niestadt commented 6 years ago

De "officiele" manier om zo'n soort waarschuwing te geven is via het "beforeunload" event, dus bijv.:

window.addEventListener("beforeunload", function (e) {
    var confirmationMessage = "Let op, u verlaat de applicatie. Doorgaan?\n\n(navigeer binnen de applicatie met behulp van de tabbladen)";

    (e || window.event).returnValue = confirmationMessage; //Gecko + IE
    return confirmationMessage; //Gecko + Webkit, Safari, Chrome etc.
});

(code komt hier vandaan, maar zijn verdere waarschuwingen en verfijningen hebben we niet nodig volgens mij)

pmasereeuw commented 6 years ago

@jan-niestadt : Dat klinkt als een heel kleine aanpassing, daar zou ik even naar kunnen kijken. Wel is het zo dat ik in mijn oplossing bij het starten van de applicatie een object in de history push, zodat je niet meteen de eerste keer uit de applicatie vliegt. Is dat ook het effect van jouw snippet?

jan-niestadt commented 6 years ago

Bij behoreunload krijg je de mogelijkheid om "nee" te zeggen, dan blijf je op de pagina. Heb je vast wel eens gezien hier en daar, pagina's die vragen of je de pagina echt wilt verlaten (bijv. als je iets aan het bewerken bent).

Dit kunnen we ook best later in een mini-update doen, dus ga vooral lekker vakantie houden. :-)

jan-niestadt commented 6 years ago

Ik heb beforeunload er nu ingebouwd. Werkt op zich goed, hoewel ik het persoonlijk irritant vind. Maar voor gebruikers die per ongeluk op Back drukken is het waarschijnlijk toch wel prettig. Indien mogelijk kunnen we in een toekomstige update alsnog Back/Forward helemaal werkend maken, dat is de beste oplossing denk ik.