1. This site uses cookies. By continuing to use this site, you are agreeing to our use of cookies. Learn More.

Fixed <div class="importantMessage"> should be <p class="importantMessage">

Discussion in 'Resolved Bug Reports' started by Vincent, Aug 12, 2012.

  1. Vincent

    Vincent Well-Known Member

    With the introduction of the Notices, the <div class="importantMessage"> stopped working.

    There still are 3 error messages in the XenForo Default Package that use <div class="importantMessage">. In order to work it should be <p class="importantMessage">.

    It's minor, but worth fixing :)

    Example of one of the 3 not-good-visible error messages: ("The specified member...")
    Screen Shot 2012-08-12 at 12.34.01.png
    Alien and Jake Bunce like this.
  2. Jake Bunce

    Jake Bunce XenForo Moderator Staff Member


    To fix, edit these three templates:

    Admin CP -> Appearance -> Templates
    > forum_view
    > member_list
    > thread_view

    Each template has a:

    <div class="importantMessage">...</div>
    That needs to be changed to:

    <p class="importantMessage">...</p>
    Or alternatively, change the CSS selector:

    Admin CP -> Appearance -> Templates -> xenforo.css

    Remove the red pieces:

    	margin: 10px 0;
    	color: @secondaryDarker;
    	background-color: @secondaryLightest;
    	text-align: center;
    	padding: 5px;
    	border-radius: 5px;
    	border: solid 1px @secondaryLight;
    p.importantMessage a
    	font-weight: bold;
    	color: @secondaryDarker;
    This is how it was in 1.0.x. I'm not sure why the change.
  3. Allan

    Allan Well-Known Member

    What is a p before the class?
  4. Jake Bunce

    Jake Bunce XenForo Moderator Staff Member

    That means it only affects "p" tags. By removing the "p" it makes it non-specific.
  5. Allan

    Allan Well-Known Member

    Thank you ;)
  6. Mike

    Mike XenForo Developer Staff Member

    I'm not sure why it was changed to require a "p", so it's just .importantMessage now.
    Vincent and Slavik like this.

Share This Page