search

gelldur / Kino-Time-Tracker

Kino Time Tracker
1 stars 0 forks source link

Okienko dialogowe helpa z wyswietleniem info #17

Closed maciej-nowak closed 11 years ago

maciej-nowak commented 11 years ago

Skorzystac z QMessageBox

maciej-nowak commented 11 years ago

Na poczatku zrobilem to z wyswietlaniem UI dla about, przypominajac sobie ze mialo to byc z QMessageBox. Wersja poprawiona wrzucona (z zachowaniem tamtej starej).

DONE!

gelldur commented 11 years ago

Super tylko musisz konwencje poprawić w tej klasie About nie używaj "_" stosujemy CamelCase. Pliki z klasami nazywamy tak samo jak klasę wewnątrz czyli pliki też z dużych aby można było korzystać z CamelCase Wiem pierdoły ale jak wejdą w nawyk to będzie naturalne i kod się bedzie łatwiej czytało.

gelldur commented 11 years ago

I taka ciekawostka nie musisz tworzyć QDialogu czyli About u nas możesz napompować obiekt QDialog ma chyba metodę setUpUi jakoś tak.

i przypomniało mi się to też przy okazji możesz dodać do konwencji (na wiki) nie nazywaj klasy About tylko np. AboutWindow , AboutDialog (ta lesza według mnie).

Dzięki takiej nazwie ktoś nawet nie musi zaglądać do pliku bo mniej więcej wie co jest w środku a tak samo About nie koniecznie powie co tam jest w środku :)

maciej-nowak commented 11 years ago

Okej, ale co do tego _ - to nie widze niczego takiego w klasie About, za to jest jest w MainWindow, o to chodzi? Co do nazw plikow, to wszystkie sa z malych liter. Poprawic wszystkie?

gelldur commented 11 years ago

slot tak nazwałeś w main window. Tak odrazu możesz poprawić wszystkie.

gelldur commented 11 years ago

Jak by co na przyszłość jak komentuje to komentuje kod który commitowałeś a 4 komenatrze wyżej github wstawia twoje commity (linkuje je) i ja je przejrzałem i patrze się na zielone twoje ewentualnie czerwone ;)

maciej-nowak commented 11 years ago

DONE