thタグのクラス設定ミス(と思われる)修正願い

フォーラム バグ報告 thタグのクラス設定ミス(と思われる)修正願い

タグ: ,

  • このトピックには8件の返信、3人の参加者があり、最後にdb_nikoにより9年、 11ヶ月前に更新されました。
9件の投稿を表示中 - 1 - 9件目 (全9件中)
  • 投稿者
    投稿
  • #69916
    db_niko
    参加者

    お世話さまです。
    とても良く出来たプラグインだと感心しながら使わせて頂いております。

    ——————————————-
    WordPress のバージョン:WP日本語版v3.9(現在の最新版)
    Welcart のバージョン:Welcart 1.3.16(現在の最新版)
    ご利用のテーマ:WP2012ベースのオリジナル
    症状を確認したブラウザ:MacOSX FireFox, Safari, Chrome
    サーバー(会社名、サービス名):さくらレンサバのビジネスプロ/PHP 5.3.28/MySQL 5.5
    SSLの利用:現在不使用
    WordPress のパーマリンク設定:/%year%/%monthnum%/%postname%/
    ——————————————–

    公開直前のショップサイトです。
    商品を購入カートに入れた段階でのクラス指定。恐縮ですが、たぶんミスだと思います。
    添付画像のように「単価」のカラム幅が見合いません。
    cssでthのwidthを修正(WPのstyle.cssに数カ所だけ!importantで強制変更して利用中)しようにも、classの指定が以下のように
    line 7 <th class=”quantity”>単価</th>
    line 8 <th class=”quantity”>数量</th>
    となっていて困っています。単価のクラスは”unit”(の筈のおつもり)ですよね?

    ざっとPHPを拝見し「order_edit_form.php」辺りかと思ったのですが場所は特定しきれず。
    お忙しい中、恐縮なのですが、ソースの中の問題と思われるので、こちらではお手上げです。
    添付画像は、FireFoxのもので、単価の金額項目の幅が潰れてしまっています。
    もちろんブラウザ関係なく同じHTMLですが、SafariとChromeは幅が潰れずに上手く表示できちゃってます(笑)Windows環境ではテストしておりません(が、差異はないですよね?)

    早めにご対応くださると、助かります。

    ===HTMLここから==========

    <div id="cart"><div class="upbutton">数量を変更した場合は必ず更新ボタンを押してください。<input type="submit" onclick="return uscesCart.upCart()" value="数量更新" name="upButton"></div><table cellspacing="0" id="cart_table">
    		<thead>
    		<tr>
    			<th class="num" scope="row">No.</th>
    			<th class="thumbnail"> </th>
    			<th>商品名</th>
    			<th class="quantity">単価</th>
    			<th class="quantity">数量</th>
    			<th class="subtotal">金額<em class="tax">(税込)</em></th>
    			<th class="stock">在庫状態</th>
    			<th class="action"> </th>
    		</tr>
    		</thead>
    		<tbody><tr>
    			<td>1</td>
    			<td><a href="http://teamkens.co.jp/shop/2014/04/tk-303-sword-1piece-backzip/"><img width="60" height="60" alt="TK-303" class="attachment-60x60" src="http://teamkens.co.jp/shop/files/2014/04/TK-303-150x150.jpg"></a></td><td class="aleft">SWORD ウェットスーツ 1ピース バックジッパー TK-303<br></td>
    			<td class="aright">¥52,920
    			</td>
    			<td><input type="text" value="1" class="quantity" name="quant[0][172][TK-303-00]"></td>
    			<td class="aright">¥52,920</td>
    			<td>在庫有り</td>
    			<td><input type="hidden" value="3" name="itemRestriction[0]">
    			<input type="hidden" value="0" name="stockid[0]">
    			<input type="hidden" value="172" name="itempostid[0]">
    			<input type="hidden" value="TK-303-00" name="itemsku[0]">
    			<input type="hidden" value="50" name="zaikonum[0][172][TK-303-00]">
    			<input type="hidden" value="52920" name="skuPrice[0][172][TK-303-00]">
    			<input type="hidden" value="" name="advance[0][172][TK-303-00]">
    			<input type="submit" value="削除" class="delButton" name="delButton[0][172][TK-303-00]">
    			</td>
    		</tr></tbody>
    		<tfoot>
    		<tr>
    			<th class="aright" scope="row" colspan="5">商品合計<em class="tax">(税込)</em></th>
    			<th class="aright">¥52,920</th>
    			<th colspan="2">&nbsp;</th>
    		</tr>
    		</tfoot>
    	</table>
    	<div class="currency_code">通貨 : 円</div></div>
    

    ===HTMLここまで==========

    以上です。どうぞよろしくお願い申し上げます。
    かしこ
    db_niko こと 太田裕也

    Attachments:
    You must be logged in to view attached files.
    #69918
    mamekichi
    参加者

    こんにちは
    wc_templates/cart/wc_cart_page.phpの下記の部分です。
    うちはせいぜい3桁なのでいままで気がつきませんでした。

    <tr>
    <th scope="row" class="num">No.</th>
    <th class="thumbnail"> </th>
    <th><?php _e('item name','usces'); ?></th>
    <th class="quantity"><?php _e('Unit price','usces'); ?></th>
    <th class="quantity"><?php _e('Quantity','usces'); ?></th>
    <th class="subtotal"><?php _e('Amount','usces'); ?><?php usces_guid_tax(); ?></th>
    <th class="stock"><?php _e('stock status','usces'); ?></th>
    <th class="action">&nbsp;</th>
    </tr>
    #69919
    mamekichi
    参加者

    あっ4桁でした。

    #69920
    db_niko
    参加者

    mamekichi 様

    ありがとうございます!ヒントを頂きました。
    長時間の作業の途中に疲れてたので、ソースをずっとたどれず、助かりました。
    商品(ウェットスーツ)の単価が万円台のものばかりなので困っておりました。

    実は、ご指摘の部分を修正しても反映されず…
    /wp-content/themes/welcart_default/wc_templates/cart/wc_cart_page.php
    頂いた上記のファイルの他をみると、
    /wp-content/themes/welcart_default/wc_templates/cart/wc_confirm_page.php
    では、Unit price のクラス指定は私の予想の
    “unit”でなく
    “price”になっていました。
    これで糸口を掴み、問題のファイルはthemesディレクトリでなくpluginsの
    /wp-content/plugins/usc-e-shop/templates/cart.php
    だとわかりました。
    こちらを直したら無事にcssの幅の指定が5%から10%になり、正常表示されました。

    Welcartさんの修正があるまで、取り敢えずこれで行けそうです。
    mamekichi 様、どうもありがとうございました。
    WelCart 様、引き続き元ファイルの修正をお願いいたします。

    #69921
    db_niko
    参加者

    mamekichi 様, WelCart 様

    合計3つのファイルのclass指定をこちらで修正しました。
    quantity → price

    /wp-content/plugins/usc-e-shop/templates/cart/cart.php

    /wp-content/plugins/usc-e-shop/theme/welcart_default/wc_templates/cart/wc_cart_page.php

    /wp-content/themes/welcart_default/wc_templates/cart/wc_cart_page.php

    同じ名前で私の方で混乱したかもしれませんが、とにかくこれで正常表示になりました。
    ありがとうございます。

    #69930
    mamekichi
    参加者

    こんにちは

    pluginsの方は触らないことが推奨されているのではないかと思います。

    仰るとおり、/wp-content/themes/welcart_default/wc_templates/cart/wc_cart_page.php を変更しただけでは変化ないですね。
    papal_vermilionがunitに変更しただけで幅が広がったので、それ以上の確認をしていませんでした。

    wc_confirm_page.phpに書いてあった、price に合わせて、wc_cart_page.php の該当部分を price に変更し、かつ、
    usces_cart.css に、対応する指定がないので、記述を足してやる必要があります。
    例えば、

    #cart .price {
    	padding-left:25px;
    	padding-right:25px;
    	line-height: 2;
    	font-size: 12px;
    	font-weight: bold;
    }

    スタイルシートは不慣れなので間違っていたり不要な記述があればご指摘ください。

    #69934
    db_niko
    参加者

    mamekichi 様

    ありがとうございます。
    本来的には、WelCart様に修正いただくのが正道かと思います。
    今後のプラグインのアップデートで上書きされてしまうかも知れません。
    しかし、4月末(つまり本日まで)が納品/検証の期日でして、取り急ぎ対応いたしました。

    CSSの方はthにwidth:10%の指定が元々あって、確認してからさぁ作業…と思ったら、すでに表示が直っていました。
    すでにCSSにpriceで設定があった事からも、prugins内のファイルの記述ミスな事が伺えます。

    と、そんな事情でありまして、わざわざご丁寧にありがとうございました。
    ちょうどGWですので、休み明けにWelCart様のご回答を待ちたいと思っております。
    かしこ

    #69946
    yskysmr
    キーマスター

    こんにちは。

    ご指摘ありがとうございます。

    wc_templates を使用していただいているのでしたら、mamekichi 様の仰るとおりの手順が正しいです。

    Welcart のプラグイン内部のソースを変更されることは、お勧めいたしておりません。また、プラグイン配下の wc_templates のテンプレートを変更されても、テーマとして機能しませんので、全く反映はされません。

    wc_templates は、オリジナルテーマで Welcart をご利用いただくユーザーの方のためのテンプレートです。

    既にオリジナルテーマ内に wc_templates を設置されている場合には、Welcart をアップデートしても、テーマ内の wc_templates は更新されません。タグ名を変更することは検討させていただきますが、オリジナルテーマ内に wc_templates を設置した場合は、usces_cart.css を設置していただいて(Welcart Shop>システム設定>「usces_cart.css の無効化」にチェックが必要です)、ご自由に編集してください。

    どうぞよろしくお願いいたします。

    #69957
    db_niko
    参加者

    yskysmr 様

    ご返信ありがとうございます。
    基本的に、初期設定の仕様のまま、少なめに手直しさせて頂いて、という使い方がモットーなのではありますが、行きがかり上あちこちファイルを見てしまいました。
    納品しているWPテーマはautomatticの2012を、けっこうカスタマイズしたものを使い、条件分岐で5つのマルチサイトを統合した形で運用しております。
    PHPファイルの中を見たら、ご報告申し上げました3つのファイルのclass指定が…、というお話でございました。

    デフォルトのCSSも、幾つかのエレメントは「! important」指定で上書きして使わせて頂いていますが、「usces_cart.css の無効化」含め、もう少し研究してみたいと存じます。
    お時間いただき、ありがとうございました。
    かしこ

9件の投稿を表示中 - 1 - 9件目 (全9件中)
  • このトピックに返信するにはログインが必要です。