BEMでのクラス名のつけ方を確認して頂きたいのですが、下記でクラス名のつけ方全般に問題はないでしょうか?

(簡易なBEMなので、すべては引き継がずという形を目指しています。)
タグも間違ってなさそうでしょうか?
いろいろな方の意見が聞きたいです。

・ソース

<div class="wrap-skew1 first-section-skew1">
<div class="wrap-skew2 first-section-skew2">
<div class="wrap-inner">
<section class=section multicolumn">
<div class="content-wrap translate3d-to-upper-left">
<h2 id="js-about-me" class="heading-primary">ABOUT&nbsp;&nbsp;ME</h2>
<p class="paragraph-oneline">あああああああああああああああああああああああああああああああああああ</p>
<p>いいいいいいいいいいいいいいいいいいいいいいいいいい。</p>
</div><!-- /.contents-wrap -->
<div class="img-wrap translate3d-lower-right">
<img class="my-image" src="img/index/myimage.png" alt="自身の写真">
</div>
</section><!-- /#about -->
</div>
</div>
</div>

下記はinnerにするべきなのでしょうか?
innerは中身ということですがいまいちはっきりしません。
<section class=section-wrap multicolumn">
<div class="content-wrap translate3d-to-upper-left">

・下記が実際の見た目です。
https://jsfiddle.net/80od313c/24/

また、BEMのclass名は「Block_key_value」で表します。とあるのですが、
keyがvalue=色が赤=color_redということでしょうか?
主語と目的語ということでしょうか?
よくわかりません。

https://app.codegrid.net/entry/bem-basic-1