site coding review!?

C

catalyst

What do You think about our local community(it is for our neighbourhood!)
website - our Trnje Hood. It's coded in HTML but rather in coding still.
There some errors in coding which i'm trying to repair to improve, but am
not managing to well, so asking for a quick look...thanks!
 
C

catalyst

catalyst said:
What do You think about our local community(it is for our neighbourhood!)
website - our Trnje Hood. It's coded in HTML but rather in coding still.
There some errors in coding which i'm trying to repair to improve, but am
not managing to well, so asking for a quick look...thanks!

www.trnje.hr
 
D

dorayme

"catalyst" <[email protected]> said:

Before going into anything, perhaps you have not noticed how users can
use different text sizes to the ones you are comfortable with. See what
happens when you alter the text size as a user.

Get every single bit of inline css style out of the html and stick it in
an external stylesheet. You can use the existing one, "stil.css".
 
T

Travis Newbury

What do You think about our local community(it is for our neighbourhood!)
website - our Trnje Hood. It's coded in HTML but rather in coding still.
There some errors in coding which i'm trying to repair to improve, but am
not managing to well, so asking for a quick look...thanks!

Well, I think it completely stinks for these reasons:

1. You misspelled almost every word on the page. it was completely
unreadable.
2. What's with all the marquees?
3. It falls apart with the first change in the font size.
4. The header pictures are too small. Choose fewer images for the
header.
5. this is going to be a nightmare to maintain, good luck
6. it is simply ugly (personal opinion ymmv)
 
C

catalyst

dorayme said:
Before going into anything, perhaps you have not noticed how users can
use different text sizes to the ones you are comfortable with. See what
happens when you alter the text size as a user.

Get every single bit of inline css style out of the html and stick it in
an external stylesheet. You can use the existing one, "stil.css".

I altered the tekst bu haven't noticed any changes that aren't on some other
sites ...!? I altered it via ALT.


Ok, i find it reasonable from the aspect of compacting, but is there some
other reasons to do so?

c.
 
C

catalyst

Blinky the Shark said:
It seems to be made for one viewport size. Yours?

--
Blinky
Killing all posts from Google Groups
The Usenet Improvement Project --> http://improve-usenet.org
Found 5/08: a free GG-blocking news *feed* --> http://usenet4all.se
About the viewport size, i don't know...i looked it from two different
computers, 1280x... and 1400x1050, on first it looked huge, and on the
second looked as i wanted to be. Don't know how to adjust it to be the same
on all sreen sizes and resolutions ?
 
B

Bergamot

John said:
What is the purpose of that thing on the right playing catch-up with the
browser's scrolling? A link to the contact page? It's annoying and
completely unnecessary.

Do what I do - disable JavaScript. :)
 
D

dorayme

"catalyst" <[email protected]> said:
I altered the tekst bu haven't noticed any changes that aren't on some other
sites ...!? I altered it via ALT.
I am not following this too well? Are you saying you are a bit reluctant
to not have the faults of other sites? What are you saying about ALT
exactly?
Ok, i find it reasonable from the aspect of compacting, but is there some
other reasons to do so?

The reason is that it is hard to troubleshoot for us. You see, we don't
care about you or your site. You have come here to a lunatic asylum
where it is every man, woman and beast for themselves. We care only
about one thing and that is having an easy time. We don't like trouble
walking in the door. If you separate your html from your CSS, this makes
it easier for us. You can take it easy and be selfish too. Do yourself a
favour and get all the inline styles outa there.
 
B

Bergamot

catalyst said:

You have an unhealthy mix of HTML presentational attributes, inline
styles and CSS. This will make maintenance a huge pain in the neck. I
suggest you

- get rid of all <font> elements, use external CSS instead
- get rid of all inline styles, use external CSS instead
- get rid of all HTML presentational attributes like align, except for
height and width attributes on <img> elements, use external CSS instead

It will make life sooooooooo much easier in the future.

Also get rid of all the excess JavaScript, on the navigation links in
particular. It does nothing except add bloat to the HTML. All the
onclick and mouseover actions can easily be done with CSS in a fraction
of the code.

Maybe you should find yourself a nice template that already has the
general layout you're looking for, and start with that. There are plenty
of free ones out there, some are over at
http://css-discuss.incutio.com/?page=CssLayouts

Sample navigation menus can be found at
http://css.maxdesign.com.au/listamatic/

You should also have a look at
http://allmyfaqs.net/faq.pl?AnySizeDesign
 
C

catalyst

Bergamot said:
You have an unhealthy mix of HTML presentational attributes, inline
Agree.

styles and CSS. This will make maintenance a huge pain in the neck. I
suggest you

- get rid of all <font> elements, use external CSS instead

Thats a whole lot of job, it means long .css sheet, is this good...I thought
it is good for .css file to be as short and brief as it can...!? I am sure i
can do that way too, but it'll take a little longer time to manage myself.
Is it possible to have to .css files loaded for the same html page, cos i
would like to have first .css for general things and second .css file for
short things like addtitional fonts etc. ?
> - get rid of all inline styles, use external CSS instead

Ok. Same as upper!
- get rid of all HTML presentational attributes like align, except for
height and width attributes on <img> elements, use external CSS instead

How on Earth would i substitute ALIGN, my every picture(i.e. IMG) has an
ALIGN. How would i do that in .css, it seems too much time consuming,
writing a css line for every picture itself.

It will make life sooooooooo much easier in the future.

I would like that toooooooo
Also get rid of all the excess JavaScript, on the navigation links in
particular. It does nothing except add bloat to the HTML. All the
onclick and mouseover actions can easily be done with CSS in a fraction
of the code.
Yes, but how to do that with .css ?

Maybe you should find yourself a nice template that already has the
general layout you're looking for, and start with that. There are plenty
of free ones out there, some are over at
http://css-discuss.incutio.com/?page=CssLayouts

Sample navigation menus can be found at
http://css.maxdesign.com.au/listamatic/

You should also have a look at
http://allmyfaqs.net/faq.pl?AnySizeDesign

Yeah, template is a good idea too, but i'll first have to manage myself in
this basic things, then i would like to start using some of design skills.
Sincerely i do not have an idea what would i add to that page thinking from
a designer viewpoint. It seems ok as an informational page, for now.

Thanks for Chit-chat!
 
R

rf

catalyst said:
Thats a whole lot of job, it means long .css sheet

Not at all.

For example you appear to want every link on the page to be
cornflowerblue (whatever that is). (also see below)

Remove all of the <font> elements color properties and insert a CSS rule:
a {color: cornflowerblue}

Do the same with font-family and font-size and pretty soon you have
Yes, but how to do that with .css ?

<a href=... class='hoveredlink'>

..hoveredlink {color: blue;}
..hoveredlink:hover {color: green;}


Lets extend this. Your navigation lives in a navigation <div> and must be
coloured blue and must be larger and must be green when hovered, however
links that are not part of the navigation (outside that navigation div)
are normal size. The navigation links are also a different font:

<div class=nav>
<a href=...>...</a>
<a href=...>...</a>
<a href=...>...</a>
</div>

<p>Other links: <a href='..'>...</a></p>

a:link {color: blue;}

a:hover {color: green;}

div.nav a {font-size: 2em; font-face: impact}

Three CSS rules (in three lines) to replace a conglomeration of font
elements. And nowhere in the HTML is anything mentioned about colour or
fonts. It's all in the CSS in exactly one place, and not exactly one
place for the HTML page in question but exactly one place for every HTML
page on the site. Want to change the colour of every link on the entire
site? Change one CSS rule.

In general CSS makes everything *simpler*.

Go over to the Zen Garden
http://www.csszengarden.com/
and look at what they do with CSS. Use the "select a design" navigation
bar.

This is not an example of what you should do, but rather what can be
done. It is also a very good example of seperating the presentational
aspects of a page from the content.

There is exactly one HTML file, containing the content. There are many
different CSS files, each presenting this same content in a very
different fashion.
 
J

Jukka K. Korpela

Scripsit rf:
Not at all.

It seems that "catalyst" has successfully trolled some people. The sad
part is that some people may actually take some of the responses
seriously.

There is usually no reason to clean up a messed-up site. Trying to do
that means _much_ more work than complete redesign and produces lousy
results.
For example you appear to want every link on the page to be
cornflowerblue (whatever that is). (also see below)

Either it's a defined color, or it is not. In CSS specifications, it
isn't.
Remove all of the <font> elements color properties and insert a CSS
rule: a {color: cornflowerblue}

Assuming you use a color value that is recognized by a browser, that's
bad advice for several reasons and generates even more damage than you
div.nav a {font-size: 2em; font-face: impact}

Let me guess... this is an attempt at anti-trolling? (I mean
specifically the font-face "property" here.)
In general CSS makes everything *simpler*.

Except things that it makes more complex or equally complex.
Go over to the Zen Garden [...]
It is also a very good example of seperating the presentational
aspects of a page from the content.

Did you read the notes about artificial and tricky markup, which is used
to make the demonstration possible at the current state of art?

If a poster is either a troll or a person who never cared to study any
introduction to the basics of first steps of elements of web authoring,
telling him, her, or it to visit the CSS Zen Garden is like given pile
of books on nuclear physics and rocket science to an illiterate.
 
B

Bergamot

rf said:
<div class=nav>
<a href=...>...</a>
<a href=...>...</a>
<a href=...>...</a>
</div>

A better idea is an unordered list. It degrades well, is more accessible
and the styling possibilities are huge.
 
R

rf

Scripsit rf:

**** off Korpela.

By mistake I have changed my _*FORGED*_ email address so I must have fallen
out of your killfile

Please put me back into it you arrogant bastard.
 
R

rf

A better idea is an unordered list. It degrades well, is more accessible
and the styling possibilities are huge.

You and I know that but the OP is still trying to get into web design 101.

As the fuckwit Korpella just pointed out, nuclear physics.



I really do wonder why I waste my bloody time looking at these newsgroups.
 
J

Jukka K. Korpela

Scripsit rf:

**** off Korpela.

It seems that some disgusting coward using the fake name "rf" just got
upset.
By mistake I have changed my _*FORGED*_ email address so I must have
fallen out of your killfile

By your manners, it seems obvious that it was not a mistake but either
ignorance comparable to, but less excusable than, the original troll's
cluelessness, or a devoted troublemaker's usual move: changing the From
field (typically, never using real data there) just to escape various
filters.

But if you really thought that other people's killfiles would protect
you from getting flamed, you're grossly mistaken. I will take the
liberty of flaming you especially in useless threads when it pleases me
and you deserve it.
The Usenet Improvement Project: http://improve-usenet.org

Did you consider making the best improvement to Usenet you could do now:
staying away from it until you are willing to behave at least by not
insulting named persons on purpose while sitting under a fake identity?
 
B

Bergamot

catalyst said:
Thats a whole lot of job, it means long .css sheet, is this good

The benefits are:
- Your HTML will be less cluttered and easier to read and maintain
- One set of rules for <a> elements in the CSS will be applied to every
<a> element on every page automatically, so if you decide to change
colors it takes only a few seconds to update one file, instead of going
through every HTML file to make <font> changes.
- An external CSS file will be cached by the browser, so only the first
page load may take an extra couple seconds. After that the browser will
use the cached file.

If you organize the CSS file well and use comments where necessary, it
should be easy enough to maintain as well.
...I thought
it is good for .css file to be as short and brief as it can...!?

That should apply to HTML, too. A little extra CSS can eliminate a lot
of excess HTML.
I am sure i
can do that way too, but it'll take a little longer time to manage myself.

Spending that extra time now will save you much, much time in the future.
Is it possible to have to .css files loaded for the same html page, cos i
would like to have first .css for general things and second .css file for
short things like addtitional fonts etc. ?

Multiple stylesheets are allowed.
How on Earth would i substitute ALIGN, my every picture(i.e. IMG) has an
ALIGN. How would i do that in .css, it seems too much time consuming,
writing a css line for every picture itself.

It's not necessary to write separate rules for every image. Set up
global rules that apply to most images, or those in a certain section,
then you only have to override those that are different.

img {
vertical-align: middle;
}
..content img {
float: left;
margin-right: 1em;
}
Yes, but how to do that with .css ?

Get rid of the table markup and use an unordered list instead. There are
many CSS rollover examples at:
 
B

Bergamot

rf said:
You and I know that but the OP is still trying to get into web design 101.

Then better to teach him good habits from the start, don't you think? ;)
 

Ask a Question

Want to reply to this thread or ask your own question?

You'll need to choose a username for the site, which only take a couple of moments. After that, you can post your question and our members will help you out.

Ask a Question

Members online

No members online now.

Forum statistics

Threads
473,744
Messages
2,569,482
Members
44,900
Latest member
Nell636132

Latest Threads

Top