本文が3行以上の投稿に「続きを読む」を表示、それ以下では非表示にしたい

Question

前提・実現したいこと

https://teratail.com/questions/306865
こちらの投稿を参考に、本文が3行以上の投稿に「続きを読む」が表示され
それ以下の行数では非表示になるお知らせを作成したいです。
指定の行数は3行にしています。

発生している問題・エラーメッセージ

1つ目の3行以上の投稿には「続きを読む」が表示され、
それ以降は3行以上でも「続きを読む」が表示されません。

例えば投稿の行数が1つ目は2行、2つ目は4行、3つ目が4行の場合
2つめの投稿にのみ「続きを読む」が表示されます。

該当のソースコード

JSは上記参考の回答のとおりに、HTMLは<h5>や<p>を追加してます。

<div class="news">
   <div class="box">
    　<h5>タイトル</h5>
    　<div class="more_txt">
       <p>1行目<br>2行目</p>
    　</div>
    　<a href="javascript:void(0)" class="txt_btn txt_open">続きを読む▼<ion-icon name="chevron-down-outline"></ion-icon></a>
    　<a href="javascript:void(0)" class="txt_btn txt_close d_none">閉じる<ion-icon name="chevron-up-outline"></ion-icon></a>
   </div>
   <div class="box">
　　　<h5>タイトル</h5>
     <div class="more_txt">
       <p>1行目<br>2行目<br>3行目<br>4行目</p>
     </div>
     <a href="javascript:void(0)" class="txt_btn txt_open">続きを読む▼<ion-icon name="chevron-down-outline"></ion-icon></a>
     <a href="javascript:void(0)" class="txt_btn txt_close d_none">閉じる<ion-icon name="chevron-up-outline"></ion-icon></a>
   </div>
   <div class="box">
     <h5>タイトル</h5>
     <div class="more_txt">
     　<p>1行目<br>2行目<br>3行目<br>4行目</p>
     </div>
     <a href="javascript:void(0)" class="txt_btn txt_open">続きを読む▼<ion-icon name="chevron-down-outline"></ion-icon></a>
     <a href="javascript:void(0)" class="txt_btn txt_close d_none">閉じる<ion-icon name="chevron-up-outline"></ion-icon></a>
   </div>
</div>

<script>
  $(document).ready(function(){
      // 続きを読む
      var $text = $('.more_txt'); // 本文
      $text.each(function(){
      var $txt_open = $(this).nextAll('.txt_open'); // 続きを読む
      var $txt_close = $(this).nextAll('.txt_close');　// 閉じる
      var lineNum = 3;
      var textHeight = $(this).height(); // テキスト全体の高さ取得
      var lineHeight = parseFloat($text.css('line-height')); // line-height取得
      var textNewHeight = lineHeight * lineNum; // 指定行数までの高さ取得
  
      if (textHeight > textNewHeight){ // 指定行数以上で作動
          $text.css({
              height: textNewHeight,
              overflow: 'hidden',
          });
  
          $txt_open.click(function(){ // 続きを読む
              $(this).next($txt_close).css('display', 'block'); // 閉じるを表示
              $(this).css('display', 'none');
              $(this).prev(".more_txt").css({
                  'height': textHeight,
                  'overflow': 'visible',
              });
          });
  
          $txt_close.click(function(){ // 閉じる
              $(this).prev($txt_open).css('display', 'block'); // 続きを読むを表示
              $(this).css('display', 'none');
              $(this).prev().prev(".more_txt").css({
                  height: textNewHeight,
                  overflow: 'hidden',
              });
          });
      } else {
          $txt_open.hide(); // 3行以下は非表示
      }
    });
  });
  </script>

.news .box{
	padding: 15px 24px 25px;
}

.news .box .more_txt {
	line-height: 1.6;
}
.news .d_none {display: none;}

試したこと

JSの高さを自動取得する部分（下記）が悪さをしているのかと思い、
heightをautoや100%にしてみるなどしましたが意味はなさそうでした。

$(this).prev(".more_txt").css({
   'height': textHeight,
   'overflow': 'visible',
});

どこが違うのかの見当もつかずお手上げ状態です。。
ご教示いただけますと幸いです。
どうぞよろしくお願いいたします。

Answer 1

 var lineHeight = parseFloat($text.css('line-height')); // line-height取得

まずこの部分が NaN を返していますよ。

面倒そうなのであまり関わりたくないのですが、
div.more_text > p となっているので、p要素の余白、line-height の割合と font-size の乗算
を調べると…

--
$('.more_txt') これは複数あるのだから

      $('.more_txt').each (function () {
        var $text = $(this); // 本文
        ...

      });

yoasobi

高さの決め打ちなら簡単なのだけれど

<style>
div.box:not(.open) div+a+a,div.box.open div+a {
  display: none;
}
div.box:not(.open) > div.more_txt {
  height: 3em;
  overflow: hidden;
}
p { margin: 0}
</style>

<div class="news">
   <div class="box">
      <h5>タイトル</h5>
      <div class="more_txt">
       <p>1行目<br>2行目<br>3行目<br>4行目</p>
      </div>
      <a href="#">続きを読む▼</a><a href="#">閉じる</a>
   </div>
   <div class="box">
     <h5>タイトル</h5>
     <div class="more_txt">
       <p>1行目<br>2行目<br>3行目<br>4行目</p>
     </div>
     <a href="#">続きを読む▼</a><a href="#">閉じる</a>
   </div>
   <div class="box">
     <h5>タイトル</h5>
     <div class="more_txt">
       <p>1行目<br>2行目<br>3行目<br>4行目</p>
     </div>
     <a href="#">続きを読む▼</a><a href="#">閉じる</a>
   </div>
</div>

<script>

function handler (event) {
  let e = event.target, div;
  if ('#' === e.getAttribute('href'))
    if (div = e.closest ('.box'))
      div.classList.toggle ('open'),
      event.preventDefault();
}
document.addEventListener ('click', handler, false);
</script>

提示されたコードに全角空白文字が含まれていました。寝ぼけていたので要らぬインデントに悩みました。

Comments

[nyrrrr]

ご回答ありがとうございます！
ここの 'line-height' が取得できてないということですね。
.more_txt に line-height のcssを追記してみましたが変わりませんでした。。
JSの書き方を変えた方が良いのでしょうか？
その場合、お手数ですがどう変えたら良いか教えていただけますと助かります。

[nyrrrr]

面倒なものに丁寧に回答いただきありがとうございます。
また全角文字失礼いたしました。
無事解決いたしました！