Pierwsza, mała strona z użyciemjQuery w pracy z plikami xml

pytanie zadane 5 lutego 2017 w JavaScript przez niezalogowany

Zechciałby ktoś ocenić kod tej strony?
http://mywww.pe.hu/
wiem że to nic specjalnego ale jest to moja pierwsza strona z użyciem jQuery.
Oczywiście jeżeli ktoś spojrzy krytycznym okiem na kod html / css też będę wdzięczny.

Stronę mam zamiar rozwijać, dodać możliwość dodawania miast etc

3 odpowiedzi

odpowiedź 5 lutego 2017 przez Kamil Naja Nałogowiec (27,410 p.)

za mały padding-top w navbarze
słabe czcionki (szeryfowe źle wyglądają, gdy się je czyta z monitora)
menu po prawej na górze - nie reaguje na hover, nie można kliknąć
Nazwa strony po lewej, też nie przenosi do main :(

komentarz 5 lutego 2017 przez imklau Nałogowiec (42,090 p.)

znacznik <head> źle umieszczony
nie ma znacznika zamykającego </input>

aaa i to co drażni to, kiedy kliknę np "Kraków" - wyświetli mi się opis, a później klikam "Wrocław" to Kraków się zamyka, a Wrocław nie otwiera ;p dopiero po drugim kliknięciu się otwiera tekst o Wrocławiu

komentarz 6 lutego 2017 przez niezalogowany

Poprawiłem chyba wszystko to co napisaliście. Troche męczyłem się z tym aby nie trzeba było na dane miasto klikać dwa razy ponnieważ szukałem najlepszego rozwiązania ale mimo wszystko nie jest zadowolony z kodu. Próbowałem używać labelu oraz continue aby nie powtarzać wywołania fonukcji / kodu. Poddałem się jednak bo to chyba nie był dobry pomysł.

PRosiłbym o ocene tego co zmieniłem, z góry dziękuje! :)

komentarz 6 lutego 2017 przez imklau Nałogowiec (42,090 p.)

teraz już ok ;)

a jeśli zamierzałbyś rozbudowywać informacje o miastach to może jednak warto by było tekst przenieść pod zdjęcia? bo jakby się pojawiło za dużo tekstu to będzie trzeba szukać później na dole zdjęć z innymi miastami :P ale na razie to chyba nie problem :P
i taka mała rzecz jeszcze, której wcześniej nie zauważyłam - miło by było, jakby kursor po najechaniu na zdjęcie zamieniał się w taki, jak masz przy "Miasta Polski" ;p

komentarz 6 lutego 2017 przez niezalogowany

mam w planie zrobić to w ten sposób że ta część z tekstem nie będzie się wysuwać tylko będzie to cośw rodzaju okna modalnego
tło będzie sie przyciemniać w informacje otrzymamy na środku ekranu

odpowiedź 5 lutego 2017 przez k.wichura Pasjonat (19,870 p.)

odpowiedź 5 lutego 2017 przez ShiroUmizake Nałogowiec (46,300 p.)

$(".city").click(function() {
			for(i=1;i<=4;i++){
				
				
				if($(this).attr("data-id") == i){
					if(zm1==0) {
						$(data).find('cities').each(function(){
							var zm22 = $(this).find('info[id="'+i+'"]').text();
							$('#cityinfo').text(zm22); 			
						});
						
						$("#cityinfo").slideToggle( "slow", function() {  }); 
						zm1=1;
						break;}
						
					else {
						$("#cityinfo").slideToggle( "slow", function() {  });
						zm1=0;
						}
				}
				

			}
			
		});
    }

Co mi się tu nie podoba?

1. Zrobiłbym to przez nodeList i obsługiwał po przez eventTarget. kodu -30%. (Jak? Poczytuj o tym ;))