CSSのみのアコーディオンメニューに枠線をつけたい

Question

このHPのHTML+CSSを使ってアコーディオンメニューを作成し、Link01・Link02の項目をそれぞれ枠線で囲みたいのですがうまくいきません。

<div class="menu">
    <label for="menu_bar01">アコーディオン１</label>
    <input type="checkbox" id="menu_bar01" class="accordion" />
    <ul id="links01">
        <li><a href="">Link01</a></li>
        <li><a href="">Link02</a></li>
        <li><a href="">Link03</a></li>
        <li><a href="">Link04</a></li>
    </ul>
    <label for="menu_bar02">アコーディオン２</label>
    <input type="checkbox" id="menu_bar02" class="accordion" />
    <ul id="links02">
        <li><a href="">Link01</a></li>
        <li><a href="">Link02</a></li>
        <li><a href="">Link03</a></li>
        <li><a href="">Link04</a></li>
    </ul>
</div>

.menu {
    max-width: 600px;
}

.menu a {
    display: block;
    padding: 15px;
    text-decoration: none;
    color: #000;
}

label {
    display: block;
    margin: 0 0 4px 0;
    padding : 15px;
    line-height: 1;
    color :#fff;
    background : green;
    cursor :pointer;
}

input {
    display: none;
}

.menu ul {
    margin: 0;
    padding: 0;
    background :#f4f4f4;
    list-style: none;
}

.menu li {
    height: 0;
    overflow: hidden;
    -webkit-transition: all 0.5s;
    -moz-transition: all 0.5s;
    -ms-transition: all 0.5s;
    -o-transition: all 0.5s;
    transition: all 0.5s;
}

#menu_bar01:checked ~ #links01 li,
#menu_bar02:checked ~ #links02 li {
    height: 54px;
    opacity: 1;
}

試したこと

.menu a {　のところに、

border: 1px solid #000000;を記載してみましたが、画像のようにlink04の下線だけがつきません。

補足情報

JQueryは使えません。
初歩的な質問になってしまいますが、ご教示願います。

Answer 1

下記でいいのでは。

/*前略*/

#menu_bar01:checked ~ #links01 li,
#menu_bar02:checked ~ #links02 li {
  height: 54px;
  opacity: 1;
  box-sizing:border-box; /*追加*/
}

/*以下追加*/
#menu_bar01:checked ~ #links01 li:last-child,
#menu_bar02:checked ~ #links02 li:last-child {
   border-bottom:1px solid #000000;
}

最後のli要素に下線を追加。
最後のli要素が下線1px分高くなるのを、box-sizing:border-box;で防ぐ。

Comments

[panisyo]

やりたかった事が出来ました、ありがとうございます！

Answer 2

overflow:hidden;になっている<li>に対して、のちにheight:54px;としているのが原因。
画像で分かると思いますが、ちゃんと表示されれば、Link01とLink02、02と03、03と04の間が、
2pxの線に見えなければおかしい。
なので、1px分hiddenになってるってことですね。

だけど、これ高さ指定しないとtransitionが効かないんですね…ということで、以下のように。

.menu a {
  display:flex;/*変更*/
  align-items:center;/*追記*/
  box-sizing:border-box;/*追記*/
  padding:0 15px;/*変更*/
  text-decoration:none;
  color:#000;
  border:1px solid #000;/*追記*/
  border-bottom:none;/*追記*/
  height:54px;/*追記*/
}
.menu li:last-child a {/*追記*/
  border-bottom:1px solid #000;
}

ただこれだと、最下部枠内の高さが1px小さくなります。
それも対応するには、

#menu_bar01:checked ~ #links01 li:last-child,
#menu_bar02:checked ~ #links02 li:last-child,
.menu li:last-child a {
  height:55px;
}

を付け足してください。

もっとスマートなやり方がありそう。

Comments

[panisyo]

色々やりかたがあるんですね、回答ありがとうございました！