Primoris Financial

  • ironzy12

    #4833

    Hey all!

    Graphene Theme looks fantastic. I have just relaunched my website using this theme and the functionality, customisation and options available are useful for any site!

    http://primorisfinancial.com.au

    I’ve had to self learn things about HTML and CSS that I had absolutely no idea had any existence, so please if anyone has any suggestions for my site, I welcome the feedback!

    Mod

    Kenneth John Odle

    #29441

    It looks really good. I have two suggestions:

    1) This is an infinitive verb chain:

    Our aim is to build an ongoing partnership to empower clients with the knowledge to take control of their financial future.

    You might want to shorten(and hence, strengthen it) to something like:

    We empower clients to take control of their financial future.

    Shorter, stronger, and it tells what you do, rather than what you aim to do. (Beware the jargon-speak!)

    2) You have a big chunk of centered text at the bottom of the page that starts “Primoris Financial is operated by xLife Pty Ltd….”. Centered text rarely looks good on the web. (And I have a list of amateur poetry sites that back up that assertion.)

    You might want to wrap that big bit of text in a <div> formatted for three columns. Something like this:

    <div style="-moz-column-count: 2; -moz-column-gap: 20px; -webkit-column-count: 2; -webkit-column-gap: 20px; column-count: 2; column-gap: 20px;">

    Add your text, and follow it with a closing </div>. You might also want to make the text a shade smaller. It would definitely be easier to read.

    (I realize the first point has nothing to do with HTML or CSS. Sorry, just can’t help it!)

    Mod

    Kenneth John Odle

    #29442

    That code does work, even in the content for posts using the HTML editor. You can see an example here:

    http://test.kjodle.net/blog/2012/06/19/testing-multicolumn-mode/

    ironzy12

    #29443

    Thanks Kenneth,

    I agree the on the non-html / css suggestion. It does flow better and I have amended! I am not the biggest fan of columns for text only, in any regard; I have certain restrictions with that disclaimer placed on me to show the text in a certain manner and in a certain place (and size!) I am considering changing that text box to an image of the text as it shows on every page and I am concerned about the ‘duplicate’ content factor, but it doesn’t present as well.

    The only issue I am struggling to fix is the removing the bullets from lists (see life insurance page). I have included the blanket ul, li { list-style-type: none; } in the stylesheet, but it has only removed the numbers across the site not the bullets, but that is a question that is about to be posted in the support forum.. (hmmm as I type I think I may change the lists from bullets to numbers to try to remove that way..)

    Mod

    Kenneth John Odle

    #29444

    I have certain restrictions with that disclaimer placed on me to show the text in a certain manner and in a certain place (and size!)

    Gotcha! (Unfortunate, but a reality, alas.)

    I am considering changing that text box to an image of the text

    I’m not a big fan of this because it is usually implemented poorly. To do it right, add a bit of antialiasing and then save it as a .png, not a .jpg. Text with a white background looks okay in a .png (most of the time) but it looks awful (most of the time) in a .jpg

    The bullet for unordered lists (which are <ul>, btw; <ol> is an ordered—i.e., numbered—list) comes from a sprite file. To change it, you’ll need something like this in your css:

    ul {
    list-style-image: none;
    }

    This page may help, but also look at the theme’s stylesheet for clues:

    http://www.w3schools.com/css/css_list.asp

    Admin

    Syahir Hakim

    #29445

    A few more suggestions:

    1. Make the video on the homepage fill the entire width of the content area

    2. Disable the “Return to Top”, or change it to just ↑

    3. You might also want to balance the number of submenu items for each top-level menu items in the footer.

Viewing 6 posts - 1 through 6 (of 6 total)

You must be logged in to reply to this topic.

Do NOT follow this link or you will be banned from the site!