[WykopX]

Hej @m__b oraz @a__s + @wykop

Trochę nie rozumiem dlaczego w kodzie HTML każdy komentarz ma id="comment-261316225" zamiast po prostu id="261316225".

Nie powinno tak być, ponieważ linki do komentarzy wyglądają w ten sposób:
https://wykop.pl/wpis/74151495/michal-biauek#261316225

Czyli maja na koncu anchor #261316225 z ID komentarza, a nie #comment-261316225
Niestety całkowicie uniemożliwia to używanie nowych selektorów CSS jak np. :target który zaznacza element HTML na podstawie anchor podanego w adresie URL za #

Zresztą właśnie dlatego, że ID ma inną wartość niż anchor w URL - przez kilka miesięcy po wprowadzeniu nowego wykopu nie działało przechodzenie na stronie do komentarza, w którym ktoś nas woła.
Programiści Wykopu wprowadzili po jakimś czasie dodatkowe funkcje w Vue, które scrollują do komentarza, ale jest to dziwny workaround, który dodatkowo obciążą i tak już powolny interfejs wykopu.
A jest to workaround do czegoś działałoby norrmalnie - po prostu - w przeglądarce, tak jak na każdej stronie.

Czy jest szansa, że:
- zmienilibyście w kodzie kontener komentarza na atrybut id="261316225"
- albo alternatywnie żeby linki powiadomień do komentarzy oraz permalinki (pod datą publikacji) do komentarzy
zmienili na http://........./#comment-261316225

Do określania typów służą klasy i w tym wypadku kazdy komentarz ma poprawnie nazwane klasy: class="entry reply" które jednoznacznie rozróżniają komentarz od wpisu,.
Dlatego "comment-" jest niepotrzebny.

#wykop #wykopchangelog #nowywykop

[WykopX]

I jeszcze przy okazji.
Na każdej stronie mikrobloga i tagów jest błąd składniowy przy wpisach, które mają jakieś komentarze.

Blok z komentarzami ma nadany atrybut id="entry-comments".

Całkowicie niepotrzebnie, ponieważ tuż obok jest atrybut klasy class="comments".
Problem polega na tym, że na jednej stronie jest więc kilkadziesiąt elementów z tym samym "id" co jest oczywiście nieprawidłowe.
Id wszystkich elementów na stronie musi być unikalne, a nie ma nic unikalnego

[Plinius]

@WykopX: wymaganie czegoś od wyklopu? xD

[ooskar]

@WykopX: id chyba nie moze zaczynac sie od liczby

[MirkoManx]

@WykopX: wiesz może z jakiego powodu zmieniło się numerowanie stron przy przeglądaniu mirko? teraz jest 'mikroblog/strona/<jakiś ciąg znaków', a dawniej było to po prostu numerowanie, przez co nie można sobie przejść sobie na szybko np. na 4 stronę

[WykopX]

@ooskar: id chyba nie moze zaczynac sie od liczby

Nie mogło kilkanaście lat temu w czasach HTML4. Teraz jak najbardziej może i id="123456" jest poprawnym atrybutem.
https://developer.mozilla.org/en-US/docs/Web/HTML/Global_attributes/id

The value must be unique amongst all the IDs in the element's home subtree and must contain at least one character. The value must not contain any space characters.

Źródło: https://www.w3.org/TR/2011/WD-html5-20110525/elements.html#the-id-attribute

[WykopX]

z jakiego powodu zmieniło się numerowanie stron przy przeglądaniu mirko

@MirkoManx: to jest feature. Hash w adresie kolejnych stron określa stronę przygotowaną specjalnie dla Ciebie.
API zwraca wpisy od razu odfiltrowane na podstawie Twojej czarnej listy tagów i użytkowników.

Paginacja w stylu
https://wykop.pl/mikroblog/strona/2
https://wykop.pl/mikroblog/strona/3

jest nadal dostępna ale tylko dla niezalogowanych użytkowników.

[MirkoManx]

to jest feature.

@WykopX: Ach, super feature. Dzięki za odpowiedź!

[toshibaas]

@WykopX: samo id w formie liczbowej to proszenie się o kłopoty przy konstrukcji wykopu :) Druga sprawa, że korzystają też z prerendera aka server side renderingu, który jak usuniesz klasę .prerender pokazuje stronę w czystej formie. Pewnie jest to cache dla optymalizacji i każdy dostaje to samo potem dopiero dzieje się cały proces w tle. Nie wiem jak wygląda to w VUE ale powinni pobawić się z hydratacją tego w odpowiedni

[WykopX]

@toshibaas

ale tak było od zawsze i pewnie nie chcieli tego zmieniać (legacy world) :D

Ale wiesz, że wykop od stycznia 2023 ma nowy kod, całkowicie przepisany od podstaw? ( ͡º ͜ʖ͡º)

Także tu nie ma mowy o zamierzchłych czasach i legacy code, prawda? Prawda? ( ͡º ͜ʖ͡º)

[toshibaas]

@WykopX: tak, tak, ale dane w bazie dalej są "legacy", więc pewnie nie chcieli nic z tym kombinować i zmieniać konstrukcji linków, ładnie ten kawałek gówienka olukrowali i jechali dalej :D

[SpokojnyLudzik]

@WykopX: coś się popsuło się.

[WykopX]

@SpokojnyLudzik co się popsulo?

[SpokojnyLudzik]

@WykopX: dziwna czarna czcionka "Główna"... "Wykopalisko"... wcześniej była biała. Tak samo słowo "Główna" było u mnie ukryte... + wyskoczyło, że mam zainstalować Wykop X Style... niby czemu, skoro mam najnowszą wersję wersji Blank?

[WykopX]

@SpokojnyLudzik

@WykopX: dziwna czarna czcionka "Główna"... "Wykopalisko"... wcześniej była biała.

Hmm... Chyba to wina nowej aktualizacji Wykop XS. Możesz sprawdzić czy przy wyłączonym XA też jest czarny kolor?

[SpokojnyLudzik]

@WykopX: Co to XA? Po wyłączeniu XS jest ok.

[WykopX]

@SpokojnyLudzik tak, chodzilo o XS. Ok, naprawie ten błąd. Na ten czas włączenie własnego koloru teksty w belce w Wykop X Style powinno chyba zadziałać

[SpokojnyLudzik]

@WykopX: Spytam z ciekawości, korzystasz w ogóle z Mirkoukrywacza?

[WykopX]

coś się popsuło się.

@SpokojnyLudzik: Zgłaszam, że naprawione ;)

https://wykop.pl/wpis/74187095/wykopx

[SpokojnyLudzik]

@WykopX: Działa ( ͡° ͜ʖ ͡°) A dałoby zrobić, aby zupełnie oddzielnie można było zwiększyć rozmiar tytułów znalezisk? (bo mi się w magiczny sposób zmniejszyły po aktualizacji, a zwiększanie tekstu powiększa mi też opisy... i gdy mam odpowiednią wielkość tytułów to opisy są za duże... jakby dało się to oddzielnie regulować, to byłoby super ;))