Telex.hu támogass banner elrejtése

[antivirtel]

Korábban a DIV-et el kellett távolítanom egy korábbi szabály hozzáadásánál, a SPAN-al is ez a helyzet? Van erről több részlet a repóban? A README, CONTRIBUTING és a sections/README-ben nem látok erre egyértelmű szabályt, csak hogy olvassak el 3 külső doksit, ami órákig tartana. Köszi

[scripthunter7]

@antivirtel A most beküldött szabály pontosan mit rejt el az oldalon?

Korábban a DIV-et el kellett távolítanom egy korábbi szabály hozzáadásánál, a SPAN-al is ez a helyzet?

Mi volt a korábbi szabály?

[antivirtel]

Hali @scripthunter7!

Egy "támogass minket!" nagyon sárga, zavaró bannert a lap alján, de ma nézem az oldalt (kikapcsolva a saját custom filtert), nincs már ott - szal lehet, hogy le lehet zárni a PR-t.

Azért örülnék, ha a SPAN & DIV kérdésre azért választ kapnék a későbbiekre nézve. Ez volt az eredeti PR és a kommented: 5b1bf2c#r1191048241

Köszi!

[scripthunter7]

Azért örülnék, ha a SPAN & DIV kérdésre azért választ kapnék a későbbiekre nézve. Ez volt az eredeti PR és a kommented: 5b1bf2c#r1191048241

@antivirtel Erre nincs kőbevésett szabály 🙂 Az adott task-tól függ. Annyi a lényeg, hogy a szabály "biztonságos" legyen, vagyis lehetőleg ne okozzon problémát, conflict-ot, stb.

Általánosságban egy jó módszer lehet, hogy kiírod a tag-et is (pl div[something]), ha csak div-eket akarsz elrejteni, de ha az attribute selectorodnak többféle tag-re is illeszkedie kell (pl div-ekre és span-okra egyidejűleg), akkor már elég, ha csak az attribute selectort írod ki (pl [something]). Sokszor előfordul az is, hogy az attribute selector már önmagában elég specifikus (pl amit küldtél is), én ilyenkor sem szoktam erőltetni a tag kirakását, mivel a szabály is picit olvashatóbb.

[scripthunter7]

lehet, hogy le lehet zárni a PR-t.

@antivirtel Rendben. Köszi a PR-t azért! Ha a jövőben megoldasz bármilyen issue-t, szívesen fogadom a PR-jeid :)