Feedback på portfolio

Det beror nog på att text-align:justify; är ärvd från #wrapper. Den fyller upp första raden med så många bilder som får plats med de satta marginalerna (a.thickbox img), sen fördelar den dem på hela bredden (800px) eftersom det skall vara just justify. På sista raden blir det aldrig en utjämning men isället normal vänsterjustering och satta marginaler.
Vill du ha jämna marginaler får du ställa text-align: left; på #img-container.

Det där med "web-safe" färger är nog en gammal relik som man inte behöver bry sig om längre.

Det tar sig! Elegant utan att bli stelt.

Bara ett par kommentarer:

Hastigheten. Vet inte hur det ser ut i Sverige, men på en utländsk lina tar det ohyggligt lång tid för bilderna att ladda (om någonsin). Om bilderna inte redan är weboptimerade skulle det kanske vara bra att göra just detta.

Likadant med den vita hinnan som kommer innan den förstorade bilden. För mig kommer sedan ingen stor bild alls, utan det lilla hjulet bara snurrar. Vet inte om det beror på seg uppkoppling eller datorn (10.4.11 med senaste Safari).

Minnet som aldrig.. Varför två punkter? Vill man skriva något outtalat brukar det vara tre. Eller är det en konstnärlig mening att man skall irritera sig på det?

Jag menade i din egen post! Hela sex stycken "två-punktare":

På hemsidan ser jag tre punkter förutom just på själva Minnet som aldrig fanns-sidan där rubriken ju är en bild!

Jag ser att du kör stenhårt med två punkter - tar det för lång tid att skriva tre? I så fall kan jag tipsa om att man kan hålla nere Alt och trycka punkt så blir det automagiskt "…" som ett enda tecken (Ellipsis - Wikipedia, the free encyclopedia).

Spara för web / Save for web brukar jag använda mig av i Photoshop. Man kan pressa storlekarna ganska rejält normalt sett.

det blir nog JÄTTE bra!

Jag tycker menyraden blir plottrig när submenyerna expanderas eftersom alla menyrubriker har samma stil, färg och tjocklek. Antingen skulle jag fixa så att den övre menyn "dimmas" eller kanske använda någon bakgrundsfärg för att tydliggöra strukturen (dvs ge expanderad huvudmeny samt alla underliggande menyval en bakgrundsfärg). I övrigt är det vackert

Jag har tyvärr (eller lyckligtvis - beroende på hur man ser det) bara tillgång till IE på jobbet. Jag har länge tänkt att jag skulle ta en titt på din sida från jobbet, men har inte kommit ihåg det väl på plats. Jag får försöka igen att komma ihåg det till i morgon...

Det där med * som bildbreddattribut reagerade jag på tidigare på att det inte var ett normalt värde. Jag skulle själv låtit bli att ange ett värde överhuvudtaget. Tom IE6 brukar skala proportionerligt om endast en dimension (längd eller bredd) är angiven.

Det finns en uppsjö med alternativ till Thickbox som inte har stagnerat i utvecklingen. Jag brukar köra med fancybox som mig veterligen, också fungerar i IE.

Och, ifall du hade missat det, skall du använda bilden som den är i full skala så behöver du inte ange någon dimension alls.

Jag hinner nog inte fördjuppa mig i indexhibit för ögonblicket. Du skulle kunna köra samma fulknep som med exandingMenu, dvs ersätta innehållet i filen jquery.thickbox.js med fancybox-plugginen. Du behöver även en js-fil för att skriva de rader som behövs för att haka på fancybox på dina bildgallerier och i princip skulle du kunna göra det i expandingMenu.js, men den skulle behöva laddas efter jquery.thickbox.js i headen.

Alternativt är om du kan lägga in en script-tag i slutet på det filer som använder thickbox/fancybox, och skriver in fancybox-haket där, så är det nog minst bäddat för strul.

Den enda skillnaden mellan dessa två är att "pack"-varianten är packad/komprimerad för att laddas ner snabbare när besökare kommer till sidan. Det är svårare att läsa och förstå koden i packat utförande, men det är väl inte aktuellt just nu.

Jag ser inte på vilken sida du har lagt till filerna, så jag har svårt att avgöra vad som gick rätt eller fel till.

Går man till thickbox's hemsida ser man att pluginen är nerlagd sen ett par år tillbaka, men när jag tittade in i din thickbox-plugin-fil såg jag att den plugin för indexhibit har visst fått sig en översyn senast i år.
Är det säkert att den inte funkar som den skall för IE (nu när du har rättat till ett par andra problem)?

Så här ser det ut på din pappas sida just nu:

css:
1. <style type='text/css'> @import url(http://www.kristoffernilsson.se/ndxz-studio/site/css/jquery.thickbox.css); </style>
2. <style type='text/css'>
a.thickbox { border: none; }
a.thickbox img { border: 3px solid #fff; margin: 0 6px 9px 0; }
</style>
3. <link rel='stylesheet' href='http://www.kristoffernilsson.se/ndxz-studio/s...' type='text/css' />

där rad 1 innehåller css för fancybox (vet du varför den är skriven som en import isf en länkad fil?), rad 2 thickbox-specifik css och rad 3 sajtens css.

js:
1 . <script type='text/javascript' src='http://www.kristoffernilsson.se/ndxz-studio/s...'></script>
2 . <script type="text/javascript" src="http://ajax.googleapis.com/ajax/libs/jquery/1..."></script>
3. <script type="text/javascript" src="/fancybox/jquery.fancybox-1.3.1.pack.js"></script>
4. <script type='text/javascript' src='http://www.kristoffernilsson.se/ndxz-studio/s...'></script>
5. <script type='text/javascript' src='http://www.kristoffernilsson.se/ndxz-studio/s...'></script>
6. <script type='text/javascript' src='http://www.kristoffernilsson.se/ndxz-studio/s...'></script>

Rad 1 o 2 gör exakt samma sak, laddar in jquery 1.4.2 (packad). Välj en av dem.
Rad 3 har fel sökväg och ger ett fel 404.
Rad 6 innehåller numera fancybox (ej packad).
Om rad 3 blir rätt, så behövs inte rad 6. Det är bättre om fancybox laddas i rad 3 för då kan du utnyttja expandingMenus.js-filen för att haka på fancybox på lämpliga element (bildlistor).

lämpligen skulle jag sätta js-filen för fancybox så här:
<script type='text/javascript' src='http://www.kristoffernilsson.se/ndxz-studio/s...'></script>

isf rad 3, och rad 6 blir överflödig (i js-blocket)

och css-mappen (css-fil + tillhörande bilder) med länk så här:

<link rel='stylesheet' href='http://www.kristoffernilsson.se/ndxz-studio/s...' type='text/css' />

isf rad 1 och 2 i css-blocket.

Kan man stänga av thickbox-pluginen i indexhibit? (då skulle rad 1 och 2 i css-blocket samt rad 6 i js-blocket ovan antagligen vara borta per automatik).

Nästa steg: använda fancybox på en sida, tex
Tidningsomslag : Kristoffer Nilsson

Bildlistan ser ut så här:

<div style='float: left; width: 900px; margin-right: 0px;'>

<div class="imagecontainer" style='float: left; width: 100px; margin-right: 15px;'>
<a class="thickbox" href="/files/gimgs/faste.jpg">
<img src="/files/gimgs/faste.jpg" height="150" alt="Fastighetsfolket"></img></a></div>

<div class="imagecontainer" style='float: left; width: 100px; margin-right: 15px;'>
<a class="thickbox" href="/files/gimgs/arbets.jpg">
<img src="/files/gimgs/arbets.jpg" height="150" alt="Arbetsterapeuten"></img></a></div>
...
</div>

Man skulle vilja ha det så här:

<div class="galleri">
<a href="/files/gimgs/faste.jpg" rel="galleri1">
<img src="/files/gimgs/faste.jpg" height="150" alt="Fastighetsfolket"/>
</a>
<a href="/files/gimgs/arbets.jpg" rel="galleri1">
<img src="/files/gimgs/arbets.jpg" height="150" alt="Arbetsterapeuten"/>
</a>
...
</div>

Då räcker det att lägga till:
$(".galleri a").fancybox();
på lämplig plats, tex i expandingMenus.js eller i slutet av själva sidan (inom script-taggar).
har du dessutom följande i css-en:

.galleri a{
 float: left;
}

så slipper du en massa element-style som du har idag.

Ett par kommentarer till den gallerian du har idag:

Det är samma bilder som används som tummisar som i full storlek: den första laddningen av sidan blir rätt tung då samtliga bilder måste laddas ner.

Varför är texten ovanför och diven med bilderna stylade som floats? De är efter varandra i sidflödet och borde klara sig utmärkt som vanliga block-element.

vad är wobblebox?
Edit: Nu hittade jag det, det är just den uppdaterade varianten av thickbox som du har kvar på din site...

Intressant (http://www.klaranilsson.com/), tack för titten

(Lite grann som man tackar då man kikat runt i en butik och går ut men inte köpt något)

LCR

Jag kunde snabbt titta igenom din sida från svärföräldrarnas IE 8. Allt så Ok ut tyckte jag. Bildvisningen med wobblebox fungerade som väntat. Möjligen ett problem: det gick inte att bläddra från bild till bild i visningsläge. "close" var enda alternativet, för att sedan öppna nästa.
Kan kolla i IE 7 först på måndag.