[SOLVED] Hide div based on database value

[laflair13]

I know there is a simple solution to this but I cannot figure it out.

Basically, if the $variable = 2 then I do not want the div to show. I tried the code below but I am getting a syntax error.

<div <?php if ($dbValue == 2) echo style='display: none'; ?>>div content</div>

[RyanReese]

@laflair13

<div <?php if ($dbValue == 2) echo style=‘display: none’; ?>>div content</div>

Try this.

<div<?php if ($dbValue == 2) echo " style='display: none';"; ?>>div content</div>

You didn’t have your echo around quotes.

You should really just add a class like “hide” instead and have .hide{display:none;} instead.

[laflair13]

Worked like a charm, I know it was something small I was missing. And I took your advice and made a class for it.

Thank You very much

[salzee]

You may want to convert inline styles to a simple CSS class to reference that instead. It’ll make your life easier later on:

.display-none {
      display:none;
}

Use like:

<div<?php if ($dbValue == 2) echo " class='display-none';"; ?>>div content</div>

Just a thought. I use to code using inline until I couldn’t anymore. Referencing CSS and their classes will keep things more consistent and easier to maintain IMO.

Good luck.

[RyanReese]

@salzee

You may want to convert inline styles to a simple CSS class to reference that instead. It’ll make your life easier later on:

I already suggested that.

@RyanReese

You should really just add a class like “hide” instead and have .hide{display:none;} instead.

And the OP has taken that suggestion.

@laflair13

And I took your advice and made a class for it.

[salzee]

@RyanReese

I already suggested that.

My fault. Sorry about that.