Hi Ahmed,
Sorry but I do not agree with these changes, they are not needed.
Using href="JAVASCRIPT:...." is a bad idea because then the link does not work at all if javascript is disabled.
It is far better to use unobtrusive javascript. If you want to wire up something to the click event of the register link for example you can do it easily and unobtrusively from layout.master like this:
<portal:RegisterLink id="RegisterLink" runat="server" CssClass="sitelink registerlink" RenderAsListItem="true" />
<script type="text/javascript">
$(document).ready(function () {
$("a.registerlink").click(function(){ ... });
});
</script>
Doing it this way means the link can still work normally if javascript is disabled.
Best,
Joe