Šablony

[frantamiz]

Dobrý večer, omlouvám se, že zde píšu česky...
Posílám tu možnou úpravu ohledně šablon.
V hlavním okně jsem přidal 2 tlačítka - vložit šablonu a editace šablon
první tlačítko otevře okno, kde se dají jen vybrat šablony.
druhé tlačítko otevře editaci šablon (vytvořit, smazat, upravit)
Vycházel jsem z kódu esmsky... Javu se teprve učím, ostatně, to už jsem psal
Nezlobte se tedy za nedostatky.
PS: Díval jsem se na kód, který se zde nakopíroval a byly tam ubrány i některé řádky, se kterými jsem já nic nedělal... Upravuje něco netbeans samo? Nebo ho mám špatně nastavené?
Děkuji.
Franta Mizera

[kparal]

Díky, nechám to tady až se ideálně najde někdo, kdo k tomu udělá review. (U sebe si to můžeš udržovat tak, že si pro to v gitu uděláš vlastní větev, a potom kdykoli se aktualizuje master větev v esmsce, tak si tu svoji vlastní větev podle ní rebasuješ. Dokud se v masteru nezmění něco zásadního, tak by to vše mělo probíhat hladce).

PS: Díval jsem se na kód, který se zde nakopíroval a byly tam ubrány i některé řádky, se kterými jsem já nic nedělal... Upravuje něco netbeans samo? Nebo ho mám špatně nastavené?

Když člověk používá grafický návrhář, tak NetBeans upravuje .form soubory, a některé části .java souborů se stejným jménem.

[frantamiz]

Ještě jsem tam něco málo upravil....
Franta Mizera

[kparal]

Ahoj, mám teď trochu času, tak jsem se rozhodl na to podívat. Musím říct, že na první patch do zdrojáků Esmsky, a na to, že se Javu učíš, tak vážně skvělá práce (a i dobře udělaný pull request). Díky. Postupně budu přidávat komentáře k jednotlivým částem kódu.

Teď jen tak obecně - některá tlačítka mám oříznutá, asi je to nativním gtk vzhledem, co používám:

Mělo by stačit nenastavovat nijak velikost tlačítek, ale nechat je automaticky přizpůsobit (v NB něco jako "set to default size").

V okně editace šablon mě hodně mate, že tlačítko Zavřít je nahoře. Dialog nemá žádný název. A šablony se nedají vytvořit s koncem řádku, což je podle mě docela zásadní chyba, to by mělo jít.

Říkám si, jestli by nebylo jednodušší mít jenom jedno okno pro ovládání šablon, místo okna pro vkládání a okna pro editaci. Co třeba okno, které vypadá podobně jako nynější okno pro editaci, ale:

• tlačítko Zavřít se přesune úplně dolů
• Upravit a Smazat se prohodí, častěji se upravuje než maže
• tlačítko Vložit do zprávy bude vlevo dole pod seznamem šablon
• pole Vytvořit... se oddělá úplně, a z tlačítka Uložit šablonu se udělá Nová šablona, umístí se úplně nahoru doprava. Po stisknutí se otevře generický dialog (ten lze vytvořit pár řádkami kódu, a nemusí se navrhovat ve form designeru v NB) s editovatelným polem - něco jako při odstraňování kontaktu, akorát editovatelné. Půjdou zadat konce řádků. To stejné při úpravě existující.
• V seznamu šablon se musí počítat s tím, že mohou obsahovat konec řádku. Takže buď zobrazovat jen první řádek, nebo nějak jinak.

Při mazání šablon by se neměl automaticky zavřít dialog editace šablon. Nevím, jestli byly nějaké technické komplikace, které způsobily, že to tak děláš.

Asi bych celou funkcionalitu šablon udělal trochu nenápadnější, není pro každého. Do toolbaru bych dal ikonu mezi Zkomprimovat zprávu a Zpět, bez popisku (jenom ikonu). Zároveň do menu Zpráva bych přidal položku Vložit šablonu mezi Zkomprimovat a Poslat. Tlačítko Vložit šablonu pod textem zprávy bych zrušil úplně.

[kparal]

Toho headeru se prosím zbav, stejně v jiných souborech nikde header nemám.

[frantamiz]

Díky za info. Budu se snažit na tom postupně něco dělat, ale zase taky podle času. Ty dvě okna jsou proto, protože jsem ještě moc nechápal listenery, teď to snad nějak půjde.:)
Jen ještě nechápu rychle tohle:
" A šablony se nedají vytvořit s koncem řádku"...
PS: Ještě jsem se snažil otevřít nový pull request na issue 312 (v SMSPanelu se zobrazí jméno i s číslem), ale zobrazily se mi tam zase soubory ze šablon, v tom gitu prostě ještě plavu. No nic, musím se učit...
Franta Mizera

[kparal]

Používej prosím více než jednopísmenkové proměnné (všude kde se to týká, nejen zde). S výjimkou i pro index. A tady zrovna to rovnou přetypuj, když to bude null, tak to u přetypování nevadí. Takže:

Temp template = (Temp) templateList.getSelectedValue();
if (template == null) {
...

[kparal]

Co se týká nových ikonek, jedna z nich je nepoužitá, a ty ostatní pocházejí odkud a pod jakou jsou licencí? Já osobně vždycky bral z Tango projektu a z Ubuntu, viz Nápověda -> Zásluhy.

[frantamiz]

Ikony jsem našel na netu pod free. Podívám se na Tango projekt. Díky.

[frantamiz]

Obdivuji Vaši rychlost procházení kódu, neuvěřitelné.

[kparal]

" A šablony se nedají vytvořit s koncem řádku"...

Teď aktuálně nejde vytvořit šablona s enterem, například:

S pozdravem,
Frantisek Mizera

ale pouze

S pozdravem, Frantisek Mizera

PS: Ještě jsem se snažil otevřít nový pull request na issue 312 (v SMSPanelu se zobrazí jméno i s číslem), ale zobrazily se mi tam zase soubory ze šablon, v tom gitu prostě ještě plavu. No nic, musím se učit...

Pro každou novou vlastnost, na které pracuješ, si vytvoř novou větev odvozenou od masteru. Takže:

$ git checkout master
$ git branch sablony master
$ git branch necojineho master
$ git checkout necojineho
$ git branch
  master
  sablony
* necojineho

A na prohlížení větví doporučuji gitg nebo tig, pokud jsi na Linuxu.

Problém je, že tys zřejmě všechny svoje změny dělal na masteru, takže si je musíš první zkopírovat:

$ git branch sablony master

a pak vyresetovat master zpět na moji verzi

$ git checkout master
$ git reset --hard origin/master

a teď můžeš dál pracovat ve svojí větvi na dalších změnách

$ git checkout sablony

[kparal]

Obdivuji Vaši rychlost procházení kódu, neuvěřitelné.

Já jsem neprošel všechno, jen to, co mě na první pohled trklo do očí. Podrobné review udělám až když se pořeší ty zásadní změny.

[frantamiz]

Dobrý večer, tak se mi podařilo (snad) dodělat Vaše doporučení do šablon (jedno okno, vyskakovací dialogy, přeuspořádání tlačítek, uložení šablon do csv).
Ikony jsou teď použité z programu, jen jedna je z Tango projektu.
Franta Mizera

[frantamiz]

Ještě malé úpravy (zapomněl jsem smazat form pro vkládání šablon.....)

[frantamiz]

Ještě se omlouvám za ten včerejší první commit. Něco jsem si provedl v NetBeans a pak jsem pracně vracel změny (proto i víc commitů).
Nicméně díky tomu jsem si ujasnil pár věcí v gitu.
Udělal jsem si v mém repozitáři větev templates, ještě ale musím nějak udělat $ git reset --hard origin/master
Musím se asi naučit tu příkazovou řádku... Nějak jsem nepřišel na to, jak to udělat přímo ze stránky... Pracuju ve windows, linux je na to asi lepší....
Díky.
Franta Mizera

[frantamiz]

Dobrý den, tak v Netbean se mi už podařil reset origin/master a funguje, jak má. Zatím ale nevím, jak to udělat přímo na stránce github.... Zkoušel jsem udělat pusch z netbeans, ale hlásí to nějakou chybu....
Už jsem uvažoval, že ten repozitář smažu a vytvořím nový se stejným názvem, ale nevím, jestli tam pak dostanu ty kódy z netbeans...
Jinak díky za všechny rady, myslím, že mne to posunulo dál.
Co se týče SMSTemplate (obaluji tam ten string), tak to asi nechám na později, až se podíváte na to, co je zde.
Díky.
Franta

[frantamiz]

Omlouvám se, překlep na close....:)

[kparal]

Ahoj, omlouvám se, vážně jsem se v posledních týdnech snažil, ale nejsem časově schopen dotáhnout tento pull request do konce. Přestávám Esmsku udržovat, až na nějaké případné triviální aktualizace bran. Pokud chceš v Esmsce nějaké nové vlastnosti, tak si je vyvíjej ve svém forku, a pokud se časem tvá verze stane podstatně lepší než tato původní, rád lidem na tvoji verzi uvedu odsud odkaz. Ještě jednou se omlouvám za komplikace, ale práce na Esmsce je pro mě již nadále časově neudržitelná. Díky za tvé pull requesty a za snahu Esmsku zlepšit.