-
Notifications
You must be signed in to change notification settings - Fork 15
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
Karta z listą dostępnych materiałów nie działa poprawnie, jeśli zamkniemy poprzednią kartę z materiałem VOD #26
Comments
Nie ma w tym nic dziwnego, przecież to do poprzedniej strony załączne są skrypty, a ta to tylko prezentacja wyników. |
Jednak do strony "prezentacyjnej" można dodać template z JS, tak jak to było np. w wersji 4.9? Czyli na początku strony z wynikami dodawany "<script>" i funkcje które odpowiadają za przyciski. |
częściowo moge u siebie zreplikować 'nie działanie przycisuku otwórz'... faktycznie kiedy zamkne wczensiejsza karte: a. zanim pojawia sie wyniki, w trakcie ich generowania.., to nic sie nie wygeneruje i nie pokaze bedzie cały czas 'porsze czekac, trwa... blababla' b. jak zamkne od razu karte wwczesijesza jak tylko pokaza sie wyniki.... to faktycznie pierwsze klikniecie powoduje totalnie nic... poza zrobienie z guzika 'otworz' szarego... ale jak na ten szary klikne to sie normlanie otwiera... edit |
Próbowałem przepisać to tak by skrypty dodawać później, niby można ale to też nie rozwiązuje tego problemu. |
nie wiem czy to gra warta zachodu.... ilu bedzie co tak błyskawicznie zamknie karte ? jak dla mnie moze zostac jak jest, ale jest to tylko i wyłącznie moja opinia :D |
Dlaczego nie rozwiązuje? Skoro funkcje są na tej karcie co przyciski, to nie wywołuje ich? |
Ostrzeżenie przed zamknięciem choć wydaje się dobrym pomysłem też nie jest w tej sytuacji możliwe do zrealizowania. Wypróbowałem chyba już wszystkie sposoby jakie znam. |
Nie wiem niestety. |
Ale nawet "Kopiuj" i "Odtwórz" nie działają? Bo z zapisywaniem to trochę bardziej skomplikowana metoda, ale dla tych dwóch nie powinno być problemu. |
Otwórz mi działa. Kopiuj i Pobierz - nie. |
A u mnie właśnie to Otwórz nie działa po zamknięciu poprzedniej karty (na Chrome). |
Dla przykładu. Jak zmienimy: To ten alert "hi" wyskakuje za każdym razem, po zamknięciu poprzedniej karty też. Myślałem, żeby w to miejsce całą funkcję wkleić. EDIT:
... ale dla kopiowania dużo innych rzeczy zostało użytych :( |
a gdyby tak jak wczensiej @zacny napisał... 'po kliknieciu na pobierz' pojawia sie info wymnagajace potwierdzenia + checkbox do zaznaczenia ' nie przypominaj mi mo tym' a w tresci 'pamietaj, aby nie zamykać macierzystej karty p ootwarciu karty z linkami' ok / anuluj i sprawa rozwiazana... zaznaczysz checkbox'a wiececj nie dostaniesz info, a tak bedzi einformowac i tyle, nie jest rozwiazanie idealne ale co zorbić ;p |
Mnie osobiście satysfakcjonuje samo "Otwórz" więc jeśli można byłoby przenieść bezpośrednio (jak na przykładzie wyżej) byłoby fajnie. Co do zamykania poprzedniej karty, niektóre strony mają takie powiadomienie że jak coś wpiszemy i próbujemy zamknąć to pojawia się ostrzeżenie. Można by to wprowadzić na takiej zasadzie, że co 1 sekundę, kiedy wygenerowana jest nowa karta, skrypt główny pisałby w niej coś (jakieś losowe ID) i sprawdzał czy poprzednie ID istnieje na tej karcie. Jeśli istnieje i jest takie samo to dalej blokowały zamykanie jej (komunikatem), a jeśli nie istnieje to nie wyświetlałby żadnego komunikatu. |
nie wiem czy rozdizelanie tylko jednej funkcji na dwa obszary ma sens pod wzgledem estetyki, wszytsko powinno byc spojne jak dla mnie... ale jak bedzie to jedyne rozwiazanie mozna to przedyskutowac :D mi osobiscie kłopot zamykania karty nie dotyka bo po prostu jej nie zamykam.... ale sobie tak pomyslałem, aby był wilk syty i owca cała... a co @zacny jakby zrobic ustawienia, gdzies w skrypcie/wtyczce, w ktorych moznabyloby zdefiniowac domyslne działaniua przycisku 'pobierz video' o co chodiz... np. domyslnie i standardowo pojawialaby si enowa karta z linkami, tak jak do tje pory, ale np. moznabyłoby ustawic, aby domyslnie z autmatu po kliknieciu na 'pobierz video' otwierała sie juz karta z linkiem video... bepzosrednio, w najwyzszej jakosci djamy na to, przeciez wtedy zamkniecie karty wczesniejszej nic jzu nie zorbi, booooo video bedzi esi ejuz odtwrazac edit. |
Spoko, tylko że powoduje problemy z wyskakującymi okienkami. |
Nie powinno i okienka nie są u mnie blokowane (Chrome, Firefox). Funkcja zostaje przecież identyczna. Jest wciśnięcie przycisku - przeglądarka wie że akcja jest zamierzona i bez problemu na to zezwala. Chyba że ogólnie chodzi o onClick. Nie znam się za bardzo na JS więc pewnie masz rację w tych kwestiach, jednak jeśli testy się udały to chyba nie przeszkadzałoby dodać. |
@HerMajestyDrMona Nie widzę tutaj dobrego wyjścia. Odcinasz skryptowi jego źródła i dziwisz się, że nie działa. To nie jest kwestia, czy nie zadziała, tylko kiedy się wywali. Pewnie różnice wynikają z implementacji silnika js w przeglądarkach. Moim zdaniem tutaj można tylko ostrzec, że coś takiego będzie miało miejsce jak ktoś tak zrobi. |
Wersja dev mi się wysypuje i znika przycisk "Pobierz", ale po modyfikacjach dodanych do zwykłej wersji: Normalnie otwiera na Chrome i Firefoxie (tak jak podczas wcześniejszych testów). Nie ma żadnych powiadomień o zablokowanych okienkach.
My to rozumiemy, ale prędzej czy później pojawią się komentarze tutaj lub na greaseforku, że "czasami nie działa", trzeba będzie wykrywać o co chodzi, tłumaczyć, odpisywać. A tak może uda się cały skrypt z funkcjami Otwórz/Zapisz/Kopiuj przenieść na tą drugą kartę. |
Właśnie sam sobie zaprzeczyłeś. Przecież tam dzieje się to samo co napisałem wyżej. To, że to się otwiera link nie wyklucza mojego zrzutu ekranu (wystarczy zamknąć link i wrócić do okna z wyborem jakości). Poza tym, co to za rozwiązanie, które działa akurat dla takiej funkcji, a dla innej już nie. Dla mnie to nie jest akceptowalne. |
Nie rozumiem tego komunikatu. Skoro okienko "zostało" zablokowane, to jakim cudem karta z materiałem normalnie się otworzyła?
Owszem, ale jeśli ma w ogóle nie działać i zmuszać do otworzenia poprzedniej karty raz jeszcze, to z dwojga złego już lepiej by zostawić chociaż 1 możliwość. Mimo wszystko zastanawiam się nad innym rozwiązaniem, a mianowicie ładowaniem "<script src=", który miałby tylko i wyłącznie funkcje otwierania, kopiowania do schowka i zapisu. Jeśli onClicki wywoływały by te funkcje to powinno być OK. |
skrypt obecnie w moim odczuciu na bardzo czytelny kod, skoro teraz wcyhdozi tyle zachodu by aimplemkenowac te zmiane to nie wiem czy autentycznie jest sens skoro nei ma dobrego, pewnego i ładnego rozwiazania... bo raczej szuflowanie opcjami przegladarek tak jak jest to np. w Fx opcja browser.tabs.closeWindowWithLastTab pewnie w gre nie wchodzi i zakłąda, ze nie moznaby było tego zreazlizowac od strony skryptu |
@HerMajestyDrMona Takie rozdzielenie do ostobnego skryptu miałoby szanse działać. Muszę przemyśleć czy można to jakoś ładnie przerobić. |
@HerMajestyDrMona wszystkich akcji (pobieranie, kopiowanie, otwieranie) wydzielić się do osobnego skryptu nie da. Zamykając główne okno nie ma możliwości używać już żadnej funkcji GM_, a tego się w żaden sposób nie obejdzie. Co za tym idzie nie działa np. możliwość bezpośredniego pobierania (moim zdaniem najfajniejsza funkcja). Dalej uważam więc, że jedyne co można zrobić, to ostrzec, że tak się stanie w tym przypadku. Nie wiem jednak jak takie ostrzeżenie można by ładnie zrealizować. Próbowałem onbeforeunlaod. Nie można tu jednak ustawić komunikatu ze swoją treścią. |
Myślę, że onbeforeunlaod w jakiejkolwiek formie (nawet bez możliwości ustawienia treści) jest lepszym rozwiązaniem. Bo w sumie te otwarte karty z listą materiałów są bezużyteczne po zamknięciu karty ze skryptem. |
To nie jest problem, problemem jest komunikat, co z tego że będzie, jak nie będzie z niego wynikało jasno o co chodzi. Dla firefoxa np. jest "Ta strona prosi o potwierdzenie decyzji jej opuszczenia — wprowadzone dane mogły nie zostać zapisane." |
Hmmm... A może:
|
no nie do końca... przynajmniej w moich testach, wszytsko zalezy od szybkosci zamkniecia karrty... jak sie zamknie karte keidy wszytsko sie wczyta, to u mnie wszytsko działa, a nawet jak nie to 2x kliknecie na guzik w zakładce z linkami i tak odpala... |
Ja potrafię, to przez tego linka dodanego do szablonu. |
OK, już wiem! |
Po przemyśleniu ja to widzę tak, że przy zamykaniu karty głównej zamyka się też karta z resultatem. Po co prezentować jakiś tam błąd, który nie jest do niczego potrzebny przeciętnemu użytkownikowi. |
Popełniłem coś takiego. Moim zdaniem jest lepiej. Zamykają się dwie zakładki razem i nic nikomu do tego czemu tak jest. |
Też dobre rozwiązanie. Ewentualnie link z komunikatu błędu mógłby się otwierać w target=_blank i dodatkowo nie dopuścić do zmiany adresu (czyli jakby ktoś ręcznie wpisał link w tą samą kartę to dopiero by się zamykała). |
Pozwolę sobie otworzyć ponownie. Udało mi się znaleźć dość proste rozwiązanie do problemów opisywanych poniżej (wyświetlanie linków w ramce na górze).
Zamiana na:
Rozwiązuje opisywane wyżej problemy. Możemy więc spokojnie edytować też:
Aby link był możliwy do kliknięcia, nie rzucając się za bardzo w oczy. Ogólnie taki jest mój pomysł na obejście tego, ale być może @zacny miałeś jeszcze jakieś inne powody do wycofania tamtych zmian? |
@HerMajestyDrMona
Przykłady: #VOD_IPLA W połączeniu z tym co napisałem wcześniej po co produkować rozwinięte rozwiązanie do błahego problemu, które działa "czasami". Czyli:
Co to takiego? |
Właściwie jakby dłużej pomyśleć to może i z punktem trzecim da się coś zrobić. |
@zacny i chciałem cos na ten temat znaleźć... ale.... https://stackoverflow.com/questions/51769201/chrome-notification-not-showing-with-tampermonkey nie sporawdzłem tego czy 'dalej not supported' ale tez by to było jakies tam rozwiązanie... wyswetlic dymek, wtedy uzytokwnik dostanie info co sie własciwie stało.... |
Ale już się rozrosła i działało bardzo sprawnie. Owszem, kodu trochę jest, ale patrząc z perspektywy użyteczności długość kodu nie ma za wiele znaczenia, zwłaszcza że w żaden sposób nie wpływa to na wydajność. Jeśli chodzi o vod.pl, to jest to chyba jedyny serwis umieszczający kolejne materiały w ramce. Co do punktu 3, to powinno się udać wziąć window.parent.url (czy jak to było
Sorry, tak jakoś mi się słowo stworzyło. Chodzi o "debloat" jak na przykład na smartfonach gdzie usuwane są pre-instalowane aplikacje. W tym wypadku chodziło o usunięcie tych nowych części z kodu (sugerowanie aby korzystać na 5.11.0 zamiast 5.11.1 przy wprowadzaniu zmian). |
Szczerze mówiąc - problemu nie ma. Miejmy też na uwadze, że duża większość osób tej poprzedniej karty nie będzie zamykać i ten błąd nie będzie pojawiać się przy każdej możliwej okazji. Więc skoro linki w błędzie umożliwiają ponowne pobranie to jest to jedynie mniej-estetyczne rozwiązanie. Nic poważnego, żeby blokować przez to całą nową funkcję. Pomijając powyższe, "window.top.location.href" niestety nie działa (cross-origin frame). Sugeruję się raczej nad tym nie skupiać. |
Zrobiłem analogiczne rozwiązanie, ale oprogramowane jest ładniej. Nie ma jakiś sztucznych tworów. |
cyfrowa.tvp.pl - patch after page update
cyfrowa.tvp.pl - patch after page update
cyfrowa.tvp.pl - patch after page update
Problem wykryty na Ipli, ale wydaje mi się, że dotyczy też innych serwisów.
Po kliknięciu "Pobierz" wyświetla się nowa karta z listą dostępnych materiałów (z bitrate, rozdzielczością, itd.).
Jeśli zamkniemy poprzednią kartę z materiałem VOD (tą na której kliknęliśmy Pobierz), to po kliknięciu np. na "OTWÓRZ" - nic się nie dzieje.
Jest to uciążliwe, gdyż kartę zamykamy szybko np. ze względu na denerwującą, zazwyczaj głośniejszą od innych dźwięków reklamę, a nad wyborem bitrate/rozdzielczości możemy się chwilę zastanawiać.
Według mnie warto byłoby przypisywać URL + akcje do tych 3 przycisków już na poziomie tworzenia tej nowej karty.
The text was updated successfully, but these errors were encountered: