read your code out loud.
If funcp has class unhidden-pr, add class hidden-pr and if funcp has class hidden-pr add unhidden-pr.
So you really should be using an else if
or just else
. You also are not removing the class.
function bt1Func() {
if ($("#funcp").hasClass('unhidden-pr')) {
$("#funcp").addClass("hidden-pr").removeClass("unhidden-pr");
} else {
$("#funcp").addClass("unhidden-pr").removeClass("hidden-pr");
}
}
.hidden-pr {
display: none;
}
.unhidden-pr {
display: flex;
}
#funcp {
text-align: center;
}
<script src="https://cdnjs.cloudflare.com/ajax/libs/jquery/3.3.1/jquery.min.js"></script>
<button onclick="bt1Func()" id="bt1" class="activate style-a">Pizza</button>
<div class="container-men">
<p id="funcp" class="hidden-pr">hi</p>
</div>
IN the end you are just reinventing toggle class
function bt1Func(){
$("#funcp").toggleClass('hidden-pr unhidden-pr');
}
.hidden-pr {
display: none;
}
.unhidden-pr {
display: flex;
}
#funcp {
text-align: center;
}
<script src="https://cdnjs.cloudflare.com/ajax/libs/jquery/3.3.1/jquery.min.js"></script>
<button onclick="bt1Func()" id="bt1" class="activate style-a">Pizza</button>
<div class="container-men">
<p id="funcp" class="hidden-pr">hi</p>
</div>
but you really only need to toggle one class if you write your CSS correctly
function bt1Func(){
$("#funcp").toggleClass('active');
}
.pr {
display: none;
}
.pr.active {
display: flex;
}
<script src="https://cdnjs.cloudflare.com/ajax/libs/jquery/3.3.1/jquery.min.js"></script>
<button onclick="bt1Func()" id="bt1" class="activate style-a">Pizza</button>
<div class="container-men">
<p id="funcp" class="pr">hi</p>
</div>