-
Notifications
You must be signed in to change notification settings - Fork 23
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
Back/forward tlacidla v prehliadaci nefunguju #22
Comments
Mozem na to pozriet ak to este niekto nerobi. |
Ako? Ja som to medzicasom fixol a trochu som aj zrefactoroval js, len este musim dokoncit refactoring navigacnych odkazov. Mam v tom pokracovat, ci uz je bug fixnuty? |
@filiplikavcan Najlepšie je separovať jednotlivé feature/bugfixy do rozličných PR, aby ti to nestálo zbytočne dlho lokálne. Teda na teraz by som odložil refaktoring odkazov a pripravil PR ak to máš hotové. Ak nie, kľudne sa do toho pustím ja. |
Zajtra to dorobim a spravim PR. |
mozeme issue zavriet? |
U mna to funguje. Akurat ak chcem ist naspat z nahladu musim kliknut 2x (FF aj Chrome, Ubuntu) |
Ano, pisem to aj v PR, ze to je issue. Dovodom je, ze robia prvy back na iframe a druhy na parent dokumente. |
hm.. mne to v chrome pod windowsom funguje.. mozno by sme to mohli pridat do otestovanych os/browser bug.. aku mas verziu os, ff a chrome? vie to niekto otestovat pod macom? |
Ja som to testoval na chrome na macu. Sent from my phone On Dec 14, 2015 9:02 PM, Ludovit Scholtz [email protected] wrote: hm.. mne to v chrome pod windowsom funguje.. mozno by sme to mohli pridat do otestovanych os/browser bug.. aku mas verziu os, ff a chrome? vie to niekto otestovat pod macom? — |
Ubuntu 14.10, Chrome 46.0.2490.86, Firefox 42.0 |
treba to opravovat? ja to neviem replikovat.. |
Ja sa na to mozem pozriet, ale asi az buduci tyzden. Podla mna to nie je kriticka vec a moze to tak zatial zostat. |
Linux Mint 17.1 |
Skusil som v Chrome a Firefoxe
The text was updated successfully, but these errors were encountered: