« 1 2
Autor Zpráva
Bubák
Profil
test http://208.77.188.166
Otestoval jsem novou verzi
http://diskuse.jakpsatweb.cz/index.php?action=vthread&forum=18&topic=7 6555#28
netušil jsem, že jde o parazitní formulář do této diskuse :o)
Jan Tvrdík
Profil
Str4wberry, Dranel – reakce na prompt
Jak jsem již výše psal, tak prompt byl experiment na žádost od Chamurappi. Dopadlo to přesně tak, jak jsem očekával – proto tam původně ani nebyl.

Dranel – reakce na regexp
Testování navrhuji nejen dle www., ale i dle .TLD.
Jak jsem již psal, je potřeba ty regulární výrazy přepsat (pokud máš čas, tak je napiš). Jsem si vědom toho, že jsou špatné.

Dranel – reakce na parazitní formulář
Ano, buhužel je – taky už jsem něco omylem poslal.

OT: Nemám rád otázky typu „je nutný ten prompt?“, protože jako programátor víš, že tam nutný není :)
Jan Tvrdík
Profil
Testujte novou verzi!

Seznam změn
– Odstraněn prompt na odkaz
– Formulář už není parazitní :)
– Integrován pokus o typografické uvozovky při citaci

To do list
1. required
– někdo schopný by měl přepsat ty regulální výrazy
– zjistit, pomocí jakého kódu jsou filtrovány "falešné obrázky" a napsat javascriptový ekvivalent nebo tu ochranu úplně vyhodit (IMO lepší žádná, než funkční jen částečně)

2. optional
– escapovat nebezpečné znaky v URL
Str4wberry
Profil
Jan Tvrdík: „Testujte novou verzi!
Výborné, děkuji moc.

Reakce na reakci na prompt:
Já tu Chamurappiho část o promptu, abych se přiznal, asi moc nepochopil.
Dranel
Profil
Jan Tvrdík
Místo http://adresa doporučuji http://www.example.com/
Jan Tvrdík
Profil
Testujte novou verzi!

Seznam změn
– nasazaní nových regulární výrazů
– odstraněna kontrola funkčnosti obrázku (pokud někdo dodá kód, který to filtruje na straně server, bude kontrola znovu obnovena)
– místo "http://adresa" se nyní vkládá "http://example.com"
– pokuď dáte citovat a není vybrán žádný text, jste upozorněni
– odkaz na citování se vkládá i v IE7
– pročištění kódu

To do list
1. required
– provést závěrečné testy funkčnosti

2. optional
– escapovat nebezpečné znaky v URI
– přidat kontrolu funkčnosti obrázků
Chamurappi
Profil
Reaguji na Str4wberryho:
je nutný ten prompt
Když z uživatele vypáčíme adresu dialogovým oknem, je větší šance, že ji správně vyplní. Motivací mého doporučení je tedy moderátorská lenost spíš než uživatelské pohodlí.

Přiznávám, že sám vymoženosti tohoto formuláře vůbec nevyužívám, takže má doporučení nevycházejí z osobní zkušenosti.


Reaguji na Jana Tvrdíka:
nasazaní nových regulární výrazů
Nejsou zbytečně složité? Není třeba testovat, jestli je zadaná adresa stoprocentně platnou URI, stačí rozpoznat, jestli bylo úmyslem uživatele zapsání adresy. Když napíšu něco, co začíná na „www.“, a kliknu na odkazové tlačítko, je hodně pravděpodobné, že chci, aby odkaz vedl na tu napsanou věc, ať už je jakákoliv.

pokud někdo dodá kód, který to filtruje na straně serveru
V současné verzi miniBB je tento regulární výraz:
/\[img\](http:\/\/([^<> \n\r\[\]&]+?)\.?(gif|jpg|jpeg|png)?)\[\/img\]/i

pokuď dáte citovat a není vybrán žádný text, jste upozorněni
Odkaz vyvolávající funkci citování je u každého příspěvku. Pokud není nic vybráno, mohl by odcitovat příspěvek, ke kterému náleží — hezky jeden řádek za druhým, každý okurzívovaný a ouvozovkovaný, vyjma prázdných řádků, řádků uvnitř <pre> a těch, co už jsou citacemi někoho jiného. Uživatel by pak odmazal, na co nechce reagovat, a ke zbytku by dopsal své reakce. Takovou vymoženost bych pravděpodobně využíval.

Další námět: Citování by si mohlo všímat toho, co cituje — zachovávat tučnost, opravovat dvojité typografické uvozovky na jednoduché.

Trošku se mi nelíbí, že funkce „Citovat“ dává uvozovky dovnitř kurzívy. Obvykle je píšu kolem, Str4wberry také.

escapovat nebezpečné znaky v URI
Mrknu na to přes víkend, jestli mě někdo nepředběhne.
DoubleThink
Profil *
Prompt bych vypustil. Málokdy píšu url ručně, většinou jej kopíruju přes schránku. Prompt (který se chová modálně) mi pak bude blokovat celý browser.
Str4wberry
Profil
Chamurappi, podle mě je špatné vyplnění adresy nejčastěji zapříčiněno tlačítkem „Odkaz“ — tedy:
1. [url=http://zadanaadresa][/url]
2. [url=http://adresa.cz]http://zadanadresa[/url]

Oba problémy by mělo Janovo řešení eliminovat.
Jan Tvrdík
Profil
Chamurappi
V současné verzi miniBB je tento regulární výraz:
/\[img\](http:\/\/([^<> \n\r\[\]&]+?)\.?(gif|jpg|jpeg|png)?)\[\/img\]/i

V tom případě nechápu jeho význam, projde jím téměř cokoliv.

Nejsou ty regulární výrazy zbytečně složité?
Možná, já je nepsal :) Pokud někdo napíše lepší, dám je tam.

Pokud není nic vybráno, mohl by se odcitovat celý příspěvek, ke kterému náleží
To je velmi dobrý nápad, teď zbývá jen vymyslet, jak to napsat.

Citování by si mohlo všímat toho, co cituje
Zde by byl trochu problém s tím, jak to napsat jednoduše.

Trošku se mi nelíbí, že funkce „Citovat“ dává uvozovky dovnitř kurzívy.
Problém je, že tato úprava si vyžádá hlubší zásah do architektury skriptu. Budu nad tím přemýšlet.
(podle mě by bylo ideální zavést tag [quote] nebo něco podobného)

Další nápady na přemýšlení
– Zvážit, zda v současné době (2008) je nutné vkládat odkaz na citování jen určitým browserům. NN4 a podobné prohlížeče snad už nikdo v praxi nepoužívá.
Jan Tvrdík
Profil
Testujte novou verzi!

Seznam změn
– Pokud není vybrán žádný text, odcituje se celý příspěvěk (doufám, že takhle nějak si to Chamurappi představoval).
– Typografické uvozovky jsou nyní vkládány vně tagů
– Odkaz na citování se nyní vkládá nezávysle na použitém prohlížeči

To do list
1. required
– provést závěrečné testy funkčnosti
2. optional
– escapovat nebezpečné znaky v URI
– přidat kontrolu funkčnosti obrázků
– při citování zachovávat formátování (teoretický nápad: vytáhnout text přes innerHTML a přes regulální výrazy nahradit html tagy za bbcode)
– odkaz na citování dát přímo do HTML (nyní ho tam vkládá JS)
– vymyslet takový regulární výraz, aby nebyl ani malý ani velký :)
Dranel
Profil
Jan Tvrdík
Jen nechápu to, proč se při citaci celého příspěvku (nic neoznačím) citovaný text (celý příspěvek) rozřádkuje a cituje po řádcích. Lepší by dle mě bylo citovat celý příspěvek jako celek, avšak se zachováním řádkování. Mimochodem při citování celého příspěvku (nic nevyberu) by bylo dobré dát před to automaticky tučně jméno autora. Jinak se mi to převelice líbí.
Jan Tvrdík
Profil
Dranel
Jen nechápu to, proč se při citaci celého příspěvku citovaný text rozřádkuje a cituje po řádcích.
Já tyhle „blbosti“ nevymýšlím (nápad pochází od Chamurappiho), já je programuji. Podle mě to má logiku. Necháš ocitovat celý příspěvěk a pak se vyjádříš postupně ke každé pasáži (takhle jsem to tedy pochopil).

V současnosti to funguje tak, že někdo tady něco plácne, já to experimentálně naprogramuji a počkám na reakci okolí.

bylo by dobré dát před to automaticky tučně jméno autora
Díky za další tip.
Dranel
Profil
Jan Tvrdík
Já tyhle „blbosti“ nevymýšlím (nápad pochází od Chamurappiho), já je programuji. Podle mě to má logiku. Necháš ocitovat celý příspěvek a pak se vyjádříš postupně ke každé pasáži (takhle jsem to tedy pochopil).
Tak to je hodně velká blbost. Protože svůj názor přeci nerozděluji do řádků... píšu ho přes celý příspěvek. Takže tuhle blbost navrhuji odstranit.
Jan Tvrdík
Profil
Mohl by se někdo zodpovědný (např. Yuhů) vyjádřit k tomu, co je potřeba k nasazení skriptu do ostrého provozu?
Str4wberry
Profil
Draneli:
Tak to je hodně velká blbost.
Ne.

Protože svůj názor přeci nerozděluji do řádků...
Ty třeba ne. Někdo ale ano.

Takže tuhle blbost navrhuji odstranit.
Já ne.
DoubleThink
Profil *
Mohl by se někdo zodpovědný (např. Yuhů) vyjádřit k tomu, co je potřeba k nasazení skriptu do ostrého provozu?
Je potřeba, abych ho já nebo Yuhů nasadil. Ergo, až dospějete k finální verzi, můžete mi změněné scripty předat.
Dranel
Profil
Str4wberry
Ne.
Ano |:o)

Ty třeba ne. Někdo ale ano.
Najdi mi prosím 10 lidí krom tebe, kteří rozdělují příspěvky tak, že co věta to řádek. Pak budu zticha. Ještě doplním, že to nesmí být reakce s citacemi - ty jsou krátké |:o)

Já ne.
Já ano |:o)
Str4wberry
Profil
Reakce na Dranela:
Janovo řešení „rozděluje“ citaci v podstatě po odstavcích.

Najdi mi prosím 10 lidí krom tebe, kteří rozdělují příspěvky tak, že co věta to řádek.
Dobrá, na příklad: vertigo4, Joker, Jan Tvrdík, Chamurappi, habendorf, Railbot, Manq, Keilew, quinux a třeba Morasino.
Dranel
Profil
Str4wberry
Ale v uvedených příkladech je přeci také více vět na jednom řádku nebo naopak věta přes více řádků. A právě proto je tato funkce, která rozřeže text na řádky, na nic.
vertigo4
Profil
Heh, nevím co se tu pořádně řeší, vidím jen, že se tu na mě odkazuje. Já rozděluji většinou nějaký delší příspěvek do odstavců... jsem tak zvyklý z weblogu...
Měsíček
Profil
Já to dělám někdy v odstavcích a někdy v řádcích a mám v tom skoro svůj systém, tak ne, aby mi ho někdo boural! ;)
DoubleThink
Profil *
Dnes byl implementován script od Jana Tvrdíka. Díky.
Ostatní prosím o betatest.
Str4wberry
Profil
Dnes byl implementován script od Jana Tvrdíka. Díky.
Výborně!
Lexter
Profil
Změny se mi moc líbí. Gratuluji.
Chamurappi
Profil
S několikaměsíčním odstupem jsem přehodnotil názor na funkci „rozcitování“.

Jejím účelem měla být příprava polotovaru, do kterého se dopíší reakce, nikoliv odcitování celého příspěvku. Bohužel ji drtivá většina lidí nepochopila. Posledních pár týdnů procházím příspěvky a likviduji rozcitované několikařádkové obludy, protože tu diskutéři z absolutně nepochopitelných důvodů chtějí citovat kompletní cizí příspěvky. Nevím, co je to za ohavný zvyk, evidentně přejatý z cizích fór.

Navrhuji zrušení (či schování) funkce rozcitování a navrácení alertu + doplnění dalšího alertu, pokud někdo chce citovat text delší než 200 znaků nebo dva řádky.
Jan Tvrdík
Profil
Chamurappi
Mělo by stačit upravit funkci insertQuote v souboru main_posts-cs-new.js
function insertQuote(el)
{
	var text = getDocumentSelection();
	if (text == '') {
		alert('Vyberte text k citaci');
	} else if (text.length > 200) {
		alert('Délka citovaného textu nesmí přesáhnout 200 znaků');
	} else {
		insertTag('i', true, text, '„', "“\r\n");
	}
}


EDIT: To blokování více řádkového textu bych zvážil, pokud by byl zájem, tak samozřejmě není problém dopsat.
Pokud by se funkce "rozcitování" měla jen schovat, tak by se dalo nastavit, že bude rozcitovávát třeba jen ze stisknutým kontrolem. Také nechám ke zvážení.
Chamurappi
Profil

Chtěl bych poprosit DoubleThinka, aby v ostré verzi skriptu „main_posts-cs-new.js“ nahradil funkci insertQuote (řádky 114 až 131) za verzi, kterou uvádí Jan Tvrdík v příspěvku #27. Už mám plné zuby opravování monstrózních citací. Před rokem jsem čtyřikrát týdně opravoval špatně vložené odkazy, nyní dvakrát až dvanáctkrát denně čistím nežádoucí citace citací citujících citace.
DoubleThink
Profil *
[#28] Hotovo

Moderátor Chamurappi: Děkuji mockrát!
« 1 2
Toto téma je uzamčeno. Odpověď nelze zaslat.