Prosiłbym o ocenę aplikacji https://szybkiepisanie.com.pli kodu. https://szybkiepisanie.com.pl Github Jestem nowy w programowaniu aplikacji mobilnych i strona jest dla mnie nauką Angular.JS-a
0
1
A tutaj wejdź żeby się sprawdzić.
- Stylistycznie brzydko wygląda zdanie, które zaczyna się od "A"
- Przed "żeby" powinien być przecinek
- W górnym menu "Wyścig" prowadzi do strony głównej.
Użytkownik tekst wynik
- Spójność przede wszystkim - albo wszystko z małych liter, albo wszystko z wielkich.
- Gdy przeszłam test, w prawym górnym logu pojawił mi się przycisk "Wyloguj", choć nigdy się nie logowałam.
https://szybkiepisanie.com.pl/#/signin
6. Czemu wykrzyknik?
7. Zarówno na tej stronie, jak i stronie rejestracji, przydałyby się jakieś odstępy pomiędzy polami formularza.
8. Link do rejestracji na tej stronie - ładniej by wyglądało, gdybyś dał standardowy tekst: "Nie posiadasz konta? Zarejestruj się"
9. Czemu tytuł jest w dodatkowym elemencie o klasie jumbotron? Po co ten dodatkowy margin?
https://szybkiepisanie.com.pl/#/practice
10. Dodatkowe przyciski na tej stronie to jakiś WTF - czemu są przyklejone do rogów? Position absolute? Blee. "Menu główne" wcale nie odnosi do menu, tylko do strony głównej. Dodatkowo są zbędne, przecież te same przyciski są w menu głównym (wystarczy pozwolić na odświeżenie strony, gdy klikamy w Trening...).
11. W menu głównym ta podstrona nazywa się "Trening", tu w przycisku mamy "Nowy wyścig", a w menu głównym "Wyścig" odnosi do strony głównej. Brak spójności w nazewnictwie bardzo razi.
Menu główne
12. "Szybkie pisanie", "Strona główna" i "Wyścig" odnoszą w to samo miejsce.
13. Zaokrąglone rogi menu głównego wyglądają dziwnie, szczególnie na stronie głównej, gdzie tabelka już takich rogów nie ma.
- Na stronie głównej szczególnie drażnią nierówne odstępy pomiędzy poszczególnymi elementami. Od navbara do pierwszego boxa masz 20px, potem między boxami 30px, a na samym dole od footera odstępu jest 10px, bo sticky footer włazi na margin.
- Masz ledwie 4 podstrony z minimalną ilością treści, a udało ci się wykorzystać h1, h2 i h3... a na stronie treningu wcale nie ma h, tylko tekst w p. Wszędzie inaczej...
console.log($scope.results[0]);
console.log($scope.results[1]);
-
Takie rzeczy to się usuwa...
-
Na stronie logowania wpisałam login: abc, zostawiłam hasło puste i kliknęłam "Zaloguj". Wyskoczył mi alert (czemu nie komunikat przy polu? alert jest bardzo nieelegancki) "Wpisz hasło", po czym przeniosło mnie do strony rejestracji :D
-
Na stronie logowania wpisałam login: abc i hasło: abc. Wyskoczył alert (brzydko), "Proszę wpisać adres email"... po czym przeniosło mnie do rejestracji :D Komunikat jest mylący, powinno raczej być: "Podany adres email jest nieprawidłowy". "Proszę wpisać..." mogłoby być, gdybym nic nie wpisała.
-
Na stronie logowania wpisałam login: [email protected] i hasło: test. Wyskoczył alert (brzydko): "There is no user record corresponding to this identifier. The user may have been deleted.". Czemu tym razem alert jest po angielsku? Oczywiście również przeniosło mnie do strony rejestracji ;)
-
Na stronie rejestracji wpisałam adres email: abc i hasło: abc. Tym razem komuniakt ze strony to "Please eneter an email address". Czemu po angielsku? Czemu inaczej, niż na stronie logowania?
-
Na stronie rejestracji jest przycisk "Zaloguj się".
-
Na stronie rejestracji wpisałam adres email: [email protected] i hasło: abc. Wyskoczył alert "Please enter a password". Nie rozumiem o co chodzi, przecież wpisałam hasło...
-
Rzuciłam okiem na kod. Nie respektujesz formatowania, olewasz wcięcia. Odmawiam czytania...
0
- Masz rację, zmienione.
- Teraz już nawet żeby nie ma :)
- To było przygotowane na póżniej, ale w sumie nie wiem kiedy dodam funkcjonalność ścigania się z innymi graczami więc na razie usuwam.
- Zmienione
- Było to spowodowane tym, że dostęp do bazy danych jest dostępny tylko po zalogowaniu dlatego jest tutaj automatyczne "Anonimowe logowanie", które nie powinno zmieniać nic na tym pasku, ale wkradł się bug.
- Żeby zachęcić ludzi do logowania się :) Już nie ma.
- Już jest odstęp.
- Racja :) dodane
- Nie pamiętam już czemu tak zrobiłem. W każdym razie już nie ma.
- Czyli nie używać absolute do takich przycisków? Przycisk do strony głównej rzeczywiście jest niepotrzebny, ale wylosowanie innego tekstu może się przydać. Pomyślę nad jakimś może graficznym przyciskiem i raczej zrobię tak, żeby sam tekst się zmieniał, a nie strona odświeżała.
11, 12 Oki, będę zwracać na to większą uwagę. - Navbar możliwe, że wygląda lepiej, ale jumbotrony są fajnie zaokrąglone moim zdaniem i nie przeszkadza mi tabelka :)
- Wow, nawet tego nie zauważyłem, że tam są różne odstępy. Zaraz się tym zajmę, a footer już jest naprawiony.
- Czy to ma znaczenie czy to są znaczniki h, p, div? Tak naprawdę dla użytkownika to nie ma znaczenia.
- No już usunąłem. Na razie na tej stronie testuję co i jak.
-
- Nad tym jeszcze nie pracowałem, tylko tymczasowe alerty były, np jak ten "Wpisz hasło" gdy haslo jest krótsze od 4 znaków. A co do przenoszenia do strony z rejestracją to ... nie wiem czemu przechodzi. Sszukałem, ale nie znalezłem przyczyny :) no, ale w końcu będę musiał to znaleźć.
- Tutaj też, alert, gdy email jest krótszy od 4 znaków. I tutaj też mnie zaskoczyłaś, bo nie wiem co to za error. Korzystam z logowania za pomocą strony Firebase.com i w ich dokumentacji znalazłem wszystkie errory jakie mogą wyskoczyć i na nie reaguję po polsku, ale jakby jakiś inny był jakimś cudem to go wypisze w alercie. I właśnie to się teraz stało.
- Nie zmieniłem na polski :-X
- Zmienione
- Ad. 17,18
- Korzystam z kombinacji Ctrl + A, Shift + Alt + F więc kod powinien byćsformatowany przed Visual Studio Code.
1
Czyli nie używać absolute do takich przycisków? Przycisk do strony głównej rzeczywiście jest niepotrzebny, ale wylosowanie innego tekstu może się przydać. Pomyślę nad jakimś może graficznym przyciskiem i raczej zrobię tak, żeby sam tekst się zmieniał, a nie strona odświeżała.
Generalnie absolute używa się w ostateczności. Zazwyczaj jest to zły pomysł, ale gdy wiesz co robisz, to oczywiście można używać. Według mnie te przyciski tak ułożone wyglądają po prostu nieestetycznie.
Czy to ma znaczenie czy to są znaczniki h, p, div? Tak naprawdę dla użytkownika to nie ma znaczenia.
Dla użytkownika nie. Dla twojego potencjalnego pracodawcy już tak >:] Aby było pięknie, ma być spójnie i tak prosto jak się da.
Korzystam z kombinacji Ctrl + A, Shift + Alt + F więc kod powinien byćsformatowany przed Visual Studio Code.
Ale ja nie zamierzam pobierać kodu, tylko weszłam sobie na stronę githuba i tam spojrzałam. Coś ci się z tym formatowaniem dzieje nie tak.
0
Mam jeszcze kilka pytań odnośnie takiego projektu.
- W tym projekcie używam logowania za pomocą zewnętrznej platformy - Firebase. To dobrze czy źle? Projekt jest stworzony w ramach nauki i ma w przyszłości posłużyć podczas starania o pracę. Czy będzie to dobrze odebrane czy jako pójście 'na łatwiznę? Podobnie z bazą danych - postawiona jest na tej platformie i mam do niej do niej przez API Firebase przez co nie używam ani JSON-a ani protokołu HTTP więc się tego nie nauczę. Spróbować napisać swój system logowania, postawić swoj serwer bazy danych i się na niego przenieść czy zostawić tak jak jest?
- Czy taki projekt jest czymś odpowiednim do pokazania( oczywiście jak go dopieszczę) podczas starania się o praktyki/staż w JS-ie/Angularze czy potrzebuję czegoś bardziej zaawansowanego? W planach bardziej związanych z back-endem do strony mam jeszcze tylko tak naprawdę tylko ściganie się z innymi graczami i nie wiem czy to wystarczy :/
- Jumbotrony na stronie mają margin-left i margin-right ustawione na auto. Wszystko pieknie, ładnie, ale jak na stronie głównej pojawia się pasek przewijania z boku to strona się przesuwa i nie wiem jak to naprawić.
- Na głównej stronie mam tabelkę aktualizującą się w miarę jak inni użytkownicy kończą pisać teksty. na ten moment tabelka aktualizuje się co 5 sekund. Jak sprawdzić wydajność takiej strony WWW? U mnie nie było różnicy podczas zmiany tej wartości, ale być może przy mniejszym przesyle danych powstają problemy. Jak to sprawdzić?
0
Skoro używasz Angulara 1 ( :( ) to go chociaż wykorzystaj, bo widzę, że służy Ci tylko do routingu. Po co Ci tam jQuery? Zamiast pisać na maksa imperatywnie i ręcznie manipulować DOMem użyj templatek i dyrektyw.
0
jak na bootstrap to smutno i pusto troszkę
0
Nie wiem o co wam chodzi. Ja widzę tylko menu (nie działające, poza blogiem) i stopkę.
Trzeba mieć MS Office żeby to działało?
Albo jakiś konkretny OS?
0
Strona jest bardzo słaba. Nie masz się czym w zasadzie chwalić.