ilch Forum » Allgemein » Design- und Projektvorstellungen » Designanpassung Angelus Mortis!

Geschlossen
  1. So. 17.07.2011 11:15 Uhr #1
    User Pic
    Robert0506 Mitglied
    Registriert seit
    24.05.2009
    Beiträge
    204
    Beitragswertungen
    5 Beitragspunkte
    Hey Jungs und Mädels!

    Hab gerade einen Kundenauftrag fertig gemacht und wollte eure Meinung zum Code wissen! Entdeckt ihr irgendwelche Fehler, die ich in der Hektik vergessen oder verdrängt habe? Was könnte man noch verbessern?
    Wie gesagt, es geht mir nur um den Code, da das Design vom Kunden als PSD vorgegeben wurde.

    Gemacht wurde:
    + komplette Anpassung in HTML und CSS
    + Erstellung einer geänderten Newsansicht
    + Erstellung der LastForum Box
    + Erstellung der LastWars Box (noch nicht zu sehen! unter testseite.pixelphysics.de wäre die LastWars Box zu sehen)
    + Erstellung des Sliders
    + Einbindung eines erweiterten PicofX Moduls inkl. Lightbox (unter dem Content)
    + Lightbox auf alle Bilder die via BBCode eingebunden werden (also auch Forum, news usw.)
    + komplette Umcodierung der Login- und Searchbox

    MfG Robert Seitz

    betroffene Homepage: angelus-mortis.com
    pixelphysics.de // Günstige Webdesigns und Anpassungen an Ilch
    0 Mitglieder finden den Beitrag gut.
  2. So. 17.07.2011 13:35 Uhr #2
    User Pic
    Leonidas74 Mitglied
    Registriert seit
    11.02.2010
    Beiträge
    13
    Beitragswertungen
    1 Beitragspunkte
    ja sieht hammer aus, hast gute arbeit gemacht
    ja was ich da verändert hätte, währe es den forum grosser machen ohne links und rechts boxes gemacht das die forum breiter ist ok
    externer Link
    0 Mitglieder finden den Beitrag gut.
  3. So. 17.07.2011 15:57 Uhr #3
    User Pic
    Kaite Mitglied
    Registriert seit
    06.01.2011
    Beiträge
    278
    Beitragswertungen
    7 Beitragspunkte
    Schöne Arbeit.
    Insgesamt kriegt es eine Note 1- von mir. (Oder A-)

    Warum?

    Gut ist: Schön und Schlicht Gehaltenes Design, Farben der Schrift und die Schriftart selber gute Auswahl.

    Schlecht: Für einen Clan finde ich die "Enterpage" am Anfang etwas "zu" unheimlich, bzw finde ich das dies gar nichts mit einem Clan zutuen hätte, aber wenn man´s mag dann ist es ja okay.

    Info: Ich weis nicht wie es bei euch ist aber bei einer Auflösung von 1024 x 768 ist die Seite
    etwas zu groß geraten. Ich jetzt hier auf meinem "Neuen Mac" ist die Seite von 1366 x 768 perfekt. Also wirklich großes und schönes Design.

    Wenn die Enterpage am Anfang nicht von dir ist dann sage es.
    Ich werde das Schlechte dann raus edieren und eine 1+ draus machen lächeln

    Mit freundlichen Grüßen,
    Lukas
    Glaube denen, die die Wahrheit suchen, und zweifle an denen, die sie gefunden haben. (Andr Gide)
    0 Mitglieder finden den Beitrag gut.
  4. So. 17.07.2011 16:01 Uhr #4
    User Pic
    WDTeamWarLord Mitglied
    Registriert seit
    17.01.2009
    Beiträge
    539
    Beitragswertungen
    46 Beitragspunkte
    @Kaite: Auch das Design ist nicht von Ihm. Erst lesen dann posten.

    Zur Arbeit. Find ich super. Hab bis jetzt keinen Fehler entdeckt.
    WDesign
    0 Mitglieder finden den Beitrag gut.
  5. So. 17.07.2011 17:51 Uhr #5
    User Pic
    Ahrtas Moderator
    Registriert seit
    17.12.2007
    Beiträge
    2.368
    Beitragswertungen
    210 Beitragspunkte
    ZitatZitat geschrieben von Kaite

    Schöne Arbeit.
    Insgesamt kriegt es eine Note 1- von mir. (Oder A-)

    Warum?

    Gut ist: Schön und Schlicht Gehaltenes Design, Farben der Schrift und die Schriftart selber gute Auswahl.

    Schlecht: Für einen Clan finde ich die "Enterpage" am Anfang etwas "zu" unheimlich, bzw finde ich das dies gar nichts mit einem Clan zutuen hätte, aber wenn man´s mag dann ist es ja okay.

    Info: Ich weis nicht wie es bei euch ist aber bei einer Auflösung von 1024 x 768 ist die Seite
    etwas zu groß geraten. Ich jetzt hier auf meinem "Neuen Mac" ist die Seite von 1366 x 768 perfekt. Also wirklich großes und schönes Design.

    Wenn die Enterpage am Anfang nicht von dir ist dann sage es.
    Ich werde das Schlechte dann raus edieren und eine 1+ draus machen lächeln

    Mit freundlichen Grüßen,
    Lukas

    Wenn man das OS von Apple auf einem Laptop installiert der auf Windows basiert, ist das kein Mac... merk dir das!


    @Robert0506
    Also mir gefällt die Umsetzung vom Slider und von der Navi.
    Welches Script hast du für den Slider und die Navi verwendet?
    0 Mitglieder finden den Beitrag gut.
  6. So. 17.07.2011 19:23 Uhr #6
    User Pic
    Robert0506 Mitglied
    Registriert seit
    24.05.2009
    Beiträge
    204
    Beitragswertungen
    5 Beitragspunkte
    Danke erstmal für euer Feedback!

    1. Die breite des Forums war so gewollt. Hab mir aber auch schon überlegt, die Forenseite auf 3/4 der Seitenbreite zu vergrößern. Also die rechte Boxenseite wegzulassen. Vllt. kann ich den Kunden davon noch überzeugen!

    2. Die Designgröße war so vorgegeben. Daran konnte ich im ersten Moment nicht viel ändern.

    3. Die Enterpage ist nicht von mir! Wurde auch nicht von mir angepasst. Die wurde vom Kunden so von der alten Homepage übernommen.

    @Arthas
    Also als Slidergrundgerüst diente mir dieser Slider:
    webdeveloperplus.com/jquery/featured-content-slider-using-jquery-ui/
    Wurde halt nur als Grundgerüst verwendet!

    Und als Menü hab ich dieses verwendet:
    externer Link

    Dieses Menü verwende ich am liebsten, da man es wirklich einfach einbinden und verwenden kann.
    pixelphysics.de // Günstige Webdesigns und Anpassungen an Ilch
    0 Mitglieder finden den Beitrag gut.
  7. So. 17.07.2011 20:46 Uhr #7
    User Pic
    Soldat53b gelöschter User
    Hey, sieht gut aus muss ich sagen. Nur mich oder eher gesagt mein vadilator stört etwas ^^: 

    Result: 2 Fehler / 103 Warnungen
line 6 column 1 - Warnung: <meta> element not empty or not closed
line 169 column 7 - Warnung: <input> element not empty or not closed
line 169 column 301 - Warnung: <br> element not empty or not closed
line 168 column 7 - Warnung: missing </form>
line 171 column 7 - Warnung: discarding unexpected </div>
line 172 column 9 - Warnung: <input> isn't allowed in <tr> elements
line 166 column 5 - Info: <tr> previously mentioned
line 173 column 9 - Warnung: <a> isn't allowed in <tr> elements
line 166 column 5 - Info: <tr> previously mentioned
line 173 column 58 - Warnung: <img> element not empty or not closed
line 174 column 7 - Fehler: discarding unexpected </form>
line 175 column 7 - Fehler: discarding unexpected </td>
line 190 column 83 - Warnung: <div> isn't allowed in <table> elements
line 190 column 31 - Info: <table> previously mentioned
line 195 column 116 - Warnung: <input> element not empty or not closed
line 203 column 1 - Warnung: <img> element not empty or not closed
line 206 column 1 - Warnung: <img> element not empty or not closed
line 209 column 1 - Warnung: <img> element not empty or not closed
line 212 column 1 - Warnung: <img> element not empty or not closed
line 333 column 9 - Warnung: <img> element not empty or not closed
line 242 column 6 - Warnung: <div> anchor "newshead_title" already defined
line 243 column 9 - Warnung: <div> anchor "news_infos" already defined
line 245 column 5 - Warnung: <div> anchor "news_content" already defined
line 277 column 9 - Warnung: <div> anchor "news_infos" already defined
line 279 column 5 - Warnung: <div> anchor "news_content" already defined
line 280 column 6 - Warnung: <div> anchor "news_content_inhalt" already defined

    (habe einige Rausgelöscht, damit hier nicht soviel gescrollt werden muss. Schau es dir am besten selbst an.)


    Sind natürlich einige ilch standard dinger bei usw.
    Aber nächstes mal vllt:
    <img /> statt <img>
    <br /> statt <br>
    <input /> statt <input>
    usw. Mögen die Browser glaube ich lieber.

    Und id's in css sind eig für nur einfache Elemete wie...
    <div id="header">...</div>
    bei sachen wie
    <div id="boxhead">
    class benutzen. Hat soweit ich weiß, was mit dem Ladeverhalten der Browser zu tun, aber nur von der Geschwindigkeit her.

    Sonst wie gesagt gut gemacht.

    PS: bei der neusten Version von Opera stimmt was in der Last Forum Box nicht.
    0 Mitglieder finden den Beitrag gut.
  8. So. 17.07.2011 21:14 Uhr #8
    User Pic
    Tr3icio gelöschter User
    Das mit den Id's und Klassen hat einen Grund! Eine Id darf nur 1 mal vergeben werden und eine Klasse mehrmals ( ID = Identifier [ Eine Id ist eine eindeutige einmalige Erkennung für etwas! ] ).
    0 Mitglieder finden den Beitrag gut.
  9. So. 17.07.2011 21:25 Uhr #9
    User Pic
    Soldat53b gelöschter User
    ZitatZitat geschrieben von Tr3icio

    Das mit den Id's und Klassen hat einen Grund! Eine Id darf nur 1 mal vergeben werden und eine Klasse mehrmals ( ID = Identifier [ Eine Id ist eine eindeutige einmalige Erkennung für etwas! ] ).


    ja, meinte ich eigentlich so. Aber bei stinkt normalem CSS und HTML funktionierts, aber sollte man nicht machen.
    0 Mitglieder finden den Beitrag gut.
  10. So. 17.07.2011 21:26 Uhr #10
    User Pic
    Tr3icio gelöschter User
    Hängt ab ob der Browser schlau genug / dumm genug ist ( eigentlich solln sich Browser ja an HTML und CSS richtig halten :-P )
    0 Mitglieder finden den Beitrag gut.
  11. Mo. 18.07.2011 15:32 Uhr #11
    User Pic
    Robert0506 Mitglied
    Registriert seit
    24.05.2009
    Beiträge
    204
    Beitragswertungen
    5 Beitragspunkte
    Danke nochmal fürs Feedback! Das mit den Klassen war ein guter Tipp! Vielen Dank!

    Was ich persönlich nicht so ganz nachvollziehen kann, ist wieso ist mein Quellcode 100% Valide wenn ich ihn per FileUpload bei W3C hochlade, der Quellcode hat aber dann 51 Fehler, wenn ich die Homepage online teste? Hat da jemand nen Tipp für mich?

    Also ich hab bei Opera keine Fehler entdeckt!


    Zuletzt modifiziert von Robert0506 am 18.07.2011 - 15:49:53
    pixelphysics.de // Günstige Webdesigns und Anpassungen an Ilch
    0 Mitglieder finden den Beitrag gut.
  12. Mo. 18.07.2011 16:19 Uhr #12
    User Pic
    Tr3icio gelöschter User
    Wenn du sie online testest hast du noch den Content! Der besteht auch nur aus HTML ^^

    Ich hoffe du hast auch dein CSS validiert^^
    0 Mitglieder finden den Beitrag gut.
  13. Mo. 18.07.2011 16:29 Uhr #13
    User Pic
    Robert0506 Mitglied
    Registriert seit
    24.05.2009
    Beiträge
    204
    Beitragswertungen
    5 Beitragspunkte
    Jo war eigentlich von Grund auf CSS 3 valide. Habs aber grad nochmal getestet... Hat sich doch noch ein Fehler eingeschlichen! Aber hab den Fehler grad ausgebessert!
    pixelphysics.de // Günstige Webdesigns und Anpassungen an Ilch
    0 Mitglieder finden den Beitrag gut.
Geschlossen

Zurück zu Design- und Projektvorstellungen

Optionen: Bei einer Antwort zu diesem Thema eine eMail erhalten