Skip to content
This repository has been archived by the owner on Jan 10, 2018. It is now read-only.

右下の通知表示が「すし」ではなく「いいね」になる #3

Closed
gaogao-9 opened this issue Nov 4, 2015 · 5 comments
Closed

Comments

@gaogao-9
Copy link
Contributor

gaogao-9 commented Nov 4, 2015

という不具合を見つけたは良いが、RT時とふぁぼ(すし)時でHTMLに指定されるクラス名が1箇所しか差がなく、しかもその1箇所がかなり深い子要素に指定されるので、CSSでのみの対応が難しそうだった。

のでプルリク飛ばせないけど、一応バグとして報告しておきます。
default

@gaogao-9
Copy link
Contributor Author

gaogao-9 commented Nov 4, 2015

同じツイートに対して、RTした時とふぁぼ(すし)した時のdiffを貼っておきます。
通知に関する情報は #spoonbill-outer 内に記述されるため、そこだけを抜粋します。

 <div id="spoonbill-outer">
-  <div class="WebToast is-actionable" data-redirect-to="notifications" id="spoonbill_1446628545954" style="display: block; opacity: 0;">
+  <div class="WebToast is-actionable" data-redirect-to="notifications" id="spoonbill_1446628914433" style="display: block; opacity: 0;">
     <div class="WebToast-box u-cf">
       <div class="WebToast-close">
         <button class="WebToast-closeButton" type="button">
           <span class="Icon Icon--close"></span>
           <span class="u-hiddenVisually">閉じる</span>
         </button>
       </div>
       <div class="WebToast-header u-cf">
         <div class="WebToast-imageBox u-floatLeft">
-          <span class="Icon Icon--retweeted Icon--small u-floatRight"></span>
+          <span class="Icon Icon--heartBadge  Icon--small u-floatRight"></span>
         </div>
-        <div class="WebToast-contentBox">リツイート済み </div>
+        <div class="WebToast-contentBox">「いいね」しました </div>
       </div>
       <div class="u-cf">
         <div class="WebToast-imageBox u-floatLeft">
           <a class="js-user-profile-link js-action-profile-avatar" tabindex="-1" href="/9_oagoag" data-user-id="3571300093">
             <img class="WebToast-avatar" width="32" height="32" src="https://pbs.twimg.com/profile_images/652916783712632832/Z09OLCLb_normal.png" alt="">
           </a>
         </div>
         <div class="WebToast-contentBox">
           <a class="WebToast-accountLink js-user-profile-link js-action-profile-name" href="/9_oagoag" data-user-id="3571300093">
             <b class="WebToast-fullname">がおっぴー(テスト中に付き文字数を長く)</b>
-          </a>さんがリツイート。
+          </a>さんがいいねしました
           <div class="WebToast-tweetExcerpt u-dir u-textBreak" dir="ltr">
             <p class="TweetTextSize  js-tweet-text tweet-text" lang="ja" data-aria-label-part="0">プルリク飛ばしてるのか寿司飛ばしてるのか途中で分からなくなった</p>
           </div>
         </div>
       </div>
     </div>
   </div>
 </div>

@mzyy94
Copy link
Owner

mzyy94 commented Nov 4, 2015

検証どうもです!CSSで階層をどう扱うかで実現できたりできなかったりする微妙なラインですな。。
いろいろ試行錯誤してみます

@mzyy94
Copy link
Owner

mzyy94 commented Nov 4, 2015

理想的な実現方法としては、
.WebToast-contentBox を visibility: hidden; にしてしまって、

.Icon--heartBadge::after {
    content: "「すし」しました";
    display: inline-block;
    position: inherit;
}

ですかねぇ(未検証)

aタグの後はちょっと手がつけられなさそうなのでhelp wantedです。

@tsubasan156
Copy link

こんにちは。面白い機能だと思いFirefoxの派生版CyberfoxにStylishというアドオンを追加して
CSSコピーして無事アイコンとプロフィール画面のいいね数がすしに変わったのはいいのですが
すしを選択してもアニメーションとマウスオーバーしたときすしと表示されません。これはもう諦めるしか無いのでしょうか?

@mzyy94
Copy link
Owner

mzyy94 commented Jan 15, 2016

@tsubasan156 興味を示していただきありがとうございます。 アニメーション関連はベンダープレフィックス付きCSS3を利用しているため、FIrefoxではうまく動かない場合もあります。
このリポジトリのフォーク先に幾つかChrome以外のブラウザへポートしたものがあるため、そちらを参考にしてみてはいかがでしょうか。

@mzyy94 mzyy94 closed this as completed Jan 10, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

3 participants