Skip to content
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

Update website's theme and wiki #22334

Merged
merged 1 commit into from
Mar 7, 2024
Merged

Conversation

hawkeye116477
Copy link
Contributor

@hawkeye116477 hawkeye116477 commented Mar 5, 2024

Pociąg Bs 5 nadjeżdża 🚂.

Oprócz tego poprawki do ciemnej strony mocy i rezygnacja z jQuery w niektórych skryptach (resztę zostawiam dla nieoficjalnej drużyny Google'a) 😄.
Niektóre biblioteki przestały być aktualizowane, więc wymieniłem je na nowy lepszy model.
Strona również będzie ponownie działać z najnowszym Hugo, gdyż skorzystałem z https://gohugo.io/render-hooks/images/, dzięki czemu canonifyURLs i relativeURLs (które być może kiedyś w przyszłej wersji usuną) w globalnej konfiguracji jest zbędne.

PS: Bot już wygenerował testowy podgląd => https://github.com/MajkiIT/polish-ads-filter/actions/runs/8158188374/artifacts/1298799353, który można odpalić np za pomocą Xamppa.

// {{end}}
// };

{{ $strings := slice "security" "ads" "anti-adblock" "social" "cookiesCategory" "annoyances" "sponsored" "e-commerce" "privacy" "selectCategory" "Polish" "foreign" "selectOrigin" "selectAdblocker" "selectFavoriteLists" "nothingFound" "ui_search" "optionSelected" "optionsSelected" "all" "clear"}}
Copy link
Contributor Author

@hawkeye116477 hawkeye116477 Mar 5, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

VS Code takie coś uznaje za błędny kod (nawet plugin Hugo nie pomaga, bo najwyraźniej działa tylko dla html), ale to taka nowoczesna składnia, tzw funkcje Hugo.

@xxcriticxx xxcriticxx merged commit 808c87f into MajkiIT:gh-pages Mar 7, 2024
1 check passed
@AroKol78
Copy link

AroKol78 commented Mar 7, 2024

o ile pamiętam podobna sytuacja miała już miejsce w wcześniejszej ver. i została wtedy naprawiona, tz. lista wychodzi poza widoczny obszar
tosamo001

dodatkowo można zmienić ikonę strony, u mnie przedstawia jako https://majkiit.github.io/polish-ads-filter/favicons/android-192x192.png (bonus nie widzę na siostrzanej stronie ikony https://polishannoyancefilters.netlify.app/favicon.ico)

@hawkeye116477
Copy link
Contributor Author

hawkeye116477 commented Mar 7, 2024

Co do favikon, po prostu nie mieliśmy pomysłu na żadną. Chcesz to zrób zgodnie z instrukcją => https://www.docsy.dev/docs/adding-content/iconsimages/#add-your-favicons i podeślij PR albo zipa z ikonami 😄 .
A co do reszty, to wydaje się być zależne od rozdzielczości, zobaczę potem co da się z tym fantem zrobić.

@AroKol78
Copy link

AroKol78 commented Mar 7, 2024

mam 1440x900, a jak zmniejszę okno przeglądarki to lista się przesuwa, ale zawsze ten sam kawałek jest zasłonięty, jakby nie była uwzględniona szerokość paska przewijania.

@AroKol78
Copy link

AroKol78 commented Mar 7, 2024

może to tylko specyficzne tylko dla mojej przeglądarki
costujest

@AroKol78
Copy link

AroKol78 commented Mar 7, 2024

dodatkowo zauważyłem produkcje tony historii - każde kombinacja zaznaczeń, plus w wyszukiwaniu zapamiętując po jednej literce.

@hawkeye116477
Copy link
Contributor Author

hawkeye116477 commented Mar 7, 2024

Z historią to nic nowego, żeby były widoczne parametry na pasku, tak musi być 😄. Jednak powinno zastępować obecny wpis, a nie dodawać kolejny, ale niestety jest błąd w przeglądarkach => https://bugzilla.mozilla.org/show_bug.cgi?id=753264, parę lat minęło i nie naprawione nadal.

Nie wiem co lepsze w takim przypadku, dorobić przycisk "zastosuj filtry" czy "wygeneruj link z przefiltrowanymi wynikami", choć pewnie druga opcja, bo inaczej to za każdym razem jak coś szukasz czy zmieniasz kategorię, trzeba byłoby klikać przycisk, a tak jedynie jak chcesz ten link, by np dodać do zakładek albo dla kogoś.

@krystian3w
Copy link
Contributor

Może się da przerobić na pushState https://stackoverflow.com/questions/17507091/replacestate-vs-pushstate?

@hawkeye116477
Copy link
Contributor Author

hawkeye116477 commented Mar 11, 2024

@krystian3w Nic to nie da, będzie działać tak samo. Tak więc jedyne wyjście to zrobić osobny przycisk „zapisz ustawienia", a w środku popup z linkiem czy coś w tym stylu.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants