olaf Posted May 28, 2012 Posted May 28, 2012 tylko cyferki do klikania? Mi bylo ciezko trafić w nie. Mam juz swoje lata ;) ogolnie spoko ale zle ze masz czas na takie rzeczy ;)
dziadek3D Posted May 28, 2012 Posted May 28, 2012 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)....:)))))))))))))
olaf Posted May 28, 2012 Posted May 28, 2012 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 ;)
dziadek3D Posted May 28, 2012 Posted May 28, 2012 pracodawca musi być i jest mądrzejszy.... to ostatni pewnik jaki się ostał.....:))) resztę wszamał pluszak Kryzys....!!!
torm Posted May 28, 2012 Author Posted May 28, 2012 a zlecenia chętnie przyjmę... :) dodam, że javascript w portfolio, też jest mojego autorstwa :)
adek Posted May 28, 2012 Posted May 28, 2012 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.
dziadek3D Posted May 28, 2012 Posted May 28, 2012 wykonasz tego fikołka.... i masz robotę w max3D.....:)))
torm Posted May 28, 2012 Author Posted May 28, 2012 hehe.. no to nic tylko brać się do roboty :) Adek - dzięki :)
Dobrotek Posted May 29, 2012 Posted May 29, 2012 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".
torm Posted May 30, 2012 Author Posted May 30, 2012 Dobrotek - dzięki :) ... myślałem o tym, w wolnej chwili dodam taką opcję Traitor - thnx :)
kots Posted May 31, 2012 Posted May 31, 2012 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
torm Posted May 31, 2012 Author Posted May 31, 2012 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 :)
kots Posted May 31, 2012 Posted May 31, 2012 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. ;)
polon Posted June 7, 2012 Posted June 7, 2012 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!
Recommended Posts
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 accountSign in
Already have an account? Sign in here.
Sign In Now