Hello and welcome to our community! Is this your first visit?
Register
Enjoy an ad free experience by logging in. Not a member yet? Register.
Results 1 to 7 of 7
  1. #1
    Regular Coder CurtWRC's Avatar
    Join Date
    May 2005
    Location
    UK
    Posts
    224
    Thanks
    9
    Thanked 1 Time in 1 Post

    Personal Blog Site

    I have been building this site over the past week:

    http://curt.rallystuff.net

    It is fully coded in ASP.NET. Some of it you won't be able to view as its members only but any comments on design and the sections visible would be much appreciated.

    Thanks,
    Curt.

  • #2
    Senior Coder
    Join Date
    Jul 2005
    Location
    UK
    Posts
    1,051
    Thanks
    6
    Thanked 13 Times in 13 Posts
    I'm not a fan of the hard edged boxy look, it's all a bit 'Windows 3.1' if you know what I mean

    I also find the text in places a little small and bunched up - is the text size in your two posts different in each one? That looks a little odd.

    Overall, I think what you need is just a touch of graphical flair. It obviously functions well enough and that was the point in the exercise, right?

  • #3
    teh Moderatorinator
    Join Date
    Sep 2004
    Location
    USA
    Posts
    2,472
    Thanks
    4
    Thanked 40 Times in 40 Posts
    Throws an exception when you click login..
    Code:
    Server Error in '/' Application.
    Argument 'Length' must be greater or equal to zero.
    Description: An unhandled exception occurred during the execution of the current web request. Please review the stack trace for more information about the error and where it originated in the code.
    
    Exception Details: System.ArgumentException: Argument 'Length' must be greater or equal to zero.
    
    Source Error:
    
    Line 105:            passinhex = passinhex + Hex(AscW(Mid(password, teller, 1))) + "-"
    Line 106:        Loop
    Line 107:        passinhex = Left(passinhex, Len(passinhex) - 1)
    Line 108:        password = passinhex
    Line 109:        Return password
    
    
    Source File: c:\Sites\Single18\curttimson\webroot\curt\index.aspx    Line: 107
    
    Stack Trace:
    
    [ArgumentException: Argument 'Length' must be greater or equal to zero.]
       Microsoft.VisualBasic.Strings.Left(String str, Int32 Length) +107
       ASP.index_aspx.encryptinhex(String password) in c:\Sites\Single18\curttimson\webroot\curt\index.aspx:107
       ASP.index_aspx.btnsubmit_Click(Object sender, EventArgs e) in c:\Sites\Single18\curttimson\webroot\curt\index.aspx:115
       System.Web.UI.WebControls.Button.OnClick(EventArgs e) +108
       System.Web.UI.WebControls.Button.System.Web.UI.IPostBackEventHandler.RaisePostBackEvent(String eventArgument) +57
       System.Web.UI.Page.RaisePostBackEvent(IPostBackEventHandler sourceControl, String eventArgument) +18
       System.Web.UI.Page.RaisePostBackEvent(NameValueCollection postData) +33
       System.Web.UI.Page.ProcessRequestMain() +1292
    
    
    Version Information: Microsoft .NET Framework Version:1.1.4322.2300; ASP.NET Version:1.1.4322.2300

  • #4
    Regular Coder CurtWRC's Avatar
    Join Date
    May 2005
    Location
    UK
    Posts
    224
    Thanks
    9
    Thanked 1 Time in 1 Post
    Quote Originally Posted by Pennimus
    I'm not a fan of the hard edged boxy look, it's all a bit 'Windows 3.1' if you know what I mean

    I also find the text in places a little small and bunched up - is the text size in your two posts different in each one? That looks a little odd.

    Overall, I think what you need is just a touch of graphical flair. It obviously functions well enough and that was the point in the exercise, right?
    Yeah, the main purpose was to test out all my skills so I haven't done much work on design. The reason I have made the two posts different sizes is because the top one is taken from the News database and the bottom is from the blog database. My news articles are going to be very short stuff in the future which will most probably be about one or two sentences just announcing recent updates. Whereas the blog posts may be very long on some occasions and I didn't want it to make the homepage too long. Do you think it would be best having a higher size font (10pt instead of 8pt) but only allow around 500 characters to be displayed followed by '...'?


    Quote Originally Posted by Brandoe85
    Throws an exception when you click login..
    Thanks for pointing this out to me. I had never noticed. I will put some validation in place now

  • #5
    Senior Coder
    Join Date
    Jul 2005
    Location
    UK
    Posts
    1,051
    Thanks
    6
    Thanked 13 Times in 13 Posts
    Whatever size you choose (10pt works for me), I think the fonts should be consistent across all posts regardless of their type - you could label them as news or blog in a far more logical manner.

    Truncating posts after a certain length is certainly the best option if you want to limit the size of the post on the homepage.

  • #6
    Regular Coder CurtWRC's Avatar
    Join Date
    May 2005
    Location
    UK
    Posts
    224
    Thanks
    9
    Thanked 1 Time in 1 Post
    I have now edited the latest blog post on the homepage to allow a maximum of 1000 characters. Anymore than this and it is cut short followed by '...', I am still working on validating it so an error doesn't appear when you click Login with nothing entered. For some reason <asp:RequiredFieldValidator> won't work, so I will try making an IF statement for it in the button submit sub-routine.

    Any more reviews would be much appreciated

  • #7
    Regular Coder CurtWRC's Avatar
    Join Date
    May 2005
    Location
    UK
    Posts
    224
    Thanks
    9
    Thanked 1 Time in 1 Post
    I have now fixed the login problem. If a user clicks on 'Login' without inputting any details an error message now appears.


  •  

    Posting Permissions

    • You may not post new threads
    • You may not post replies
    • You may not post attachments
    • You may not edit your posts
    •