Jump to content

Recommended Posts

  • Replies 15
  • Created
  • Last Reply

Top Posters In This Topic

Posted

tylko cyferki do klikania? Mi bylo ciezko trafić w nie. Mam juz swoje lata ;)

ogolnie spoko ale zle ze masz czas na takie rzeczy ;)

Posted

trochę to miniaturowe... ale może rzeczywiście nie wiadomo na czy to będzie obecnie wyświetlone...:))

więcej standardów niż rozsądku... teraz wokoło...!!! kiedyś był monitor 15' i HTML....wszystko było jasne...:))

 

Pani Olu.... to może coś zamówić albo dać mu robotę... zdolny jest....!!!

a tacy zawsze mają większe problemy.... daleko nie patrząc (mówię o sobie)....:)))))))))))))

Posted

wywołany do tablicy powiem, że niestety zbyt pazerny jestem. Poza tym zle sie pracuje dla kogoś co wszystko wie lepiej ;)

Swoją drogą zlecenie na zaprojektowanie czegokolwiek miałem może ostatnie z rok temu. Takie poważne pow. 1dnia pracy. Więc nawet nie wiem jak to teraz wygląda i skąd brać taką robotę oraz czy ktokolwiek jej potrzebuje ;)

Posted

Bardzo fajne torm. Ładnie się to prezentuje.

Jedynie również trochę niewygodne te numerki. Ale możnaby to obejść. Masz jquery, dodaj może zmianę tych numerków prac za pomcą kółka myszki. Na środkowym elemencie kółko myszki może zmieniać prace z danej kategorii. Nie powinno być trudnej bo większość funkcji już masz. Tylko podpiąć to pod zdarzenia. Ale ładnie.

Posted

Torm, eleganckie takie porfolio ;) Co do boksu pełniącego funkcję właściwą portfolio, przydałaby się opcja powiększenia prac, takie jak laye i prace w dziale "Druk".

Posted

Hej Torm!

 

Wizualnie jest spoko. :) Przydałoby się popracować nad tym o czym wspomniał @adek Nawigacja strzałkami oraz "scrollem" w myszce byłaby naprawdę fajna.

 

A co do kodu. :)

 

Dalej operujesz wyłącznie divami co nie jest dobrym rozwiązaniem. Przynajmniej logo mogłoby być w

:

 

TOMB - Tomasz Bieńkowski

 

Brakuje też , atrybuty alt w są puste. Tzn. - ok. - Twoja sprawa. Jeśli chcesz utrudnić wyszukiwarkom robotę... ;) Używasz też kilku identyfikatorów o tej samej nazwie (np. id="img1" powtarza się kilka razy). TO BŁĄD. MOŻE PROWADZIĆ DO NIEPOPRAWNEGO DZIAŁANIA STRONY. Używaj klas w takich przypadkach.

 

W CSS często używasz np: "div.logo", "ul.menu", itp. Jedyne co "div" robi w "div.klasa" to spowalnianie działania selektora. Oczywiście czasami zachodzi potrzeba rozróżnienia elementów np.: "div.off" i "section.off" - klasa może oznaczać co innego w takim przypadku. Nie ma to jednak miejsca w przypadku Twojej strony w związku z tym polecam poprawę selektorów. Zupełnym nonsensem jest używanie np. "li#tel" zamiast po prostu "#tel". IDENTYFIKATOR MOŻE BYĆ TYLKO JEDEN NA DANEJ STRONIE, ograniczanie go selektorem elementu nie ma uzasadnienia.

 

Na koniec JS. Dwie sprawy, które od razu rzucają się w oczy. Nie keszujesz obiektów jQuery ($) co wpływa negatywnie na wydajność skryptu. Za każdym razem gdy używasz $('#drukContainer'), a używasz wieeeele razy, jQuery będzie wykonywało nowy obiekt (nie, nie sprowadza się to tylko do wyszukania elementu tak jak w CSS :P). Stąd najlepszym rozwiązaniem jest "zapisanie" obiektu w zmiennej np.:

 

var $internetContainer = $('#internetContainer');

// później możesz sobie używać (a nie wytwarzać od nowa) zapisany w zmiennej element np:

if($internetContainer.hasClass('deactive')){
$internetContainer.animate({
		marginLeft: '269px'
	}, 1000, 'easeInQuint', function() {
		// Animation complete.
	});
	$internetContainer.removeClass('deactive');
	$internetContainer.addClass('active');
}
}

 

Druga sprawa to używanie oldschoolwego onclick w HTMLu. :) Rozumiem - działa, aaaale... Można to zrobić lepiej (zwłaszcza jeśli masz jQuery na pokładzie). :)

 

Weźmy np.:

 

</pre>
<ul class="imgChanger">
1
2
3
4
5
6
7
</ul

 

Zamiast tego sugeruję odkurzyć HTML:

 

</pre>
<ul class="imgChanger">
1
2
3
4
5
6
7
</ul

 

A w JS dodać:

 

// proszę zwrócić uwagę: inaczej niż w CSS ul.imgChanger zadziała szybciej niż .imgChanger - taki smaczek :)
// http://hungred.com/useful-information/jquery-optimization-tips-and-tricks/
// find jest duuużo szybsze niż $('ul.imgChanger li') // http://api.jquery.com/category/traversing/
// używam alternatywy dla .click - efekt ten sam // http://api.jquery.com/on/

$('ul.imgChanger').find('li').on('click', function() {

// $(this) = element który został kliknięty
// $(this).index() = numer elementu (od góry) 
// UWAGA: pierwszy element = 0, drugi = 1, trzeci = 2, itd.
// dlatego dodaję 1 - w ten sposób będzie to zgodne z nazewnictwem które przyjąłeś
shiftImageInternet($(this).index()+1);

});

 

Nie będę się już więcej rozwlekał. ;) Podsumowując: projekt fajny, kod (zwłaszcza HTML) - niestety słabo.

 

Pozdrawiam,

 

Artur

Posted

kots wiem, że kod jest dziurawy... ale jak na razie strona nie ma być wyszukiwana w google... nie takie były założenia... :) raczej ma służyć do wysyłania zainteresowanym :)

Posted

Popraw tylko te zmultiplikowane id żeby się któremuś zainteresowanemu stronka nie rozjechała. :P Większość przeglądarek powinna to ogarnąć, ale mogą być problemy. Serio, serio.

 

A co do kodu - wiem, że stać Cię na więcej. Nie usprawiedliwiaj się tylko do roboty. ;)

Posted

ważne są prace a te są na poziomie, a sam lay folio... jeśli nie przeszkadza to już jest dobrze. Torm, bez kompleksów działaj w temacie, rozsyłaj i wiedz że zlecenia powinny się pojawić bo prace są dobre! powodzenia!

Create an account or sign in to comment

You need to be a member in order to leave a comment

Create an account

Sign up for a new account in our community. It's easy!

Register a new account

Sign in

Already have an account? Sign in here.

Sign In Now



×
×
  • Create New...

Important Information

We are using cookies. Read about our Privacy Policy