質問をすることでしか得られない、回答やアドバイスがある。

15分調べてもわからないことは、質問しよう!

ただいまの
回答率

89.21%

CSS, 画像の可変がききません。

解決済

回答 2

投稿 編集

  • 評価
  • クリップ 0
  • VIEW 1,343

ishigaki16

score 10

前提・実現したいこと

既存のHTMLサイトをレスポンシブ対応化しているのですが、
画像サイズが変わりません。
以下に貼付けますが、どのような問題点があると想定できるでしょうか。
当方、まだコーダーとして初心者であるため、
疑問点などありましたら迅速に回答致しますので、
どうぞよろしくお願いいたします。

該当のソースコード

<div class="top">
        <img src="img/imagebanner.png" width="1178" height="390" alt=""/>
        </div>
        <div class="digicatalog">
        <img src="img/dicatalogbtn.png" width="334" height="70" alt=""/>
        </div>      
</header>
@charset "UTF-8";
/* CSS Document */

/* pc */
@media screen and (min-width: 961px) {
ul, li {
    list-style: none;
}

h2{
    margin-top: 80px;
}

img {
max-width : 100% ;
height : auto ;
}

body { 
    -webkit-text-size-adjust : 100% ; 
}

.header {
    width: 80%;
    margin: 0 auto;
    width: 1181px;
}

.top {
    position:relative;
    width: 1181px;
    top: 0;
}

.digicatalog {
    position: absolute;
    width: 1181px;
    top: 255px;
    left: 820px;    
}

/* tablet  */
@media only screen and (min-width: 641px) and (max-width: 960px) {
ul, li {
    list-style: none;
}

h2{
    margin-top: 80px;
}

img {
max-width : 100% ;
height : auto ;
}

body { 
    -webkit-text-size-adjust : 100% ; 
}

.header {
    width: 80%;
    margin: 0 auto;
    width: 1181px;
}

.top {
    position:relative;
    width: 1181px;
    top: 0;
}

.digicatalog {
    position: absolute;
    width: 1181px;
    top: 255px;
    left: 820px;    
}

/* smartPhone */
@media screen and (max-width: 640px) {
ul, li {
    list-style: none;
}

h2{
    margin-top: 80px;
}

img {
max-width : 100% ;
height : auto ;
}

body { 
    -webkit-text-size-adjust : 100% ; 
}

.header {
    width: 80%;
    margin: 0 auto;
    width: 1181px;
}

.top {
    position:relative;
    width: 1181px;
    top: 0;
}

.digicatalog {
    position: absolute;
    width: 1181px;
    top: 255px;
    left: 820px;    
}
  • 気になる質問をクリップする

    クリップした質問は、後からいつでもマイページで確認できます。

    またクリップした質問に回答があった際、通知やメールを受け取ることができます。

    クリップを取り消します

  • 良い質問の評価を上げる

    以下のような質問は評価を上げましょう

    • 質問内容が明確
    • 自分も答えを知りたい
    • 質問者以外のユーザにも役立つ

    評価が高い質問は、TOPページの「注目」タブのフィードに表示されやすくなります。

    質問の評価を上げたことを取り消します

  • 評価を下げられる数の上限に達しました

    評価を下げることができません

    • 1日5回まで評価を下げられます
    • 1日に1ユーザに対して2回まで評価を下げられます

    質問の評価を下げる

    teratailでは下記のような質問を「具体的に困っていることがない質問」、「サイトポリシーに違反する質問」と定義し、推奨していません。

    • プログラミングに関係のない質問
    • やってほしいことだけを記載した丸投げの質問
    • 問題・課題が含まれていない質問
    • 意図的に内容が抹消された質問
    • 過去に投稿した質問と同じ内容の質問
    • 広告と受け取られるような投稿

    評価が下がると、TOPページの「アクティブ」「注目」タブのフィードに表示されにくくなります。

    質問の評価を下げたことを取り消します

    この機能は開放されていません

    評価を下げる条件を満たしてません

    評価を下げる理由を選択してください

    詳細な説明はこちら

    上記に当てはまらず、質問内容が明確になっていない質問には「情報の追加・修正依頼」機能からコメントをしてください。

    質問の評価を下げる機能の利用条件

    この機能を利用するためには、以下の事項を行う必要があります。

質問への追記・修正、ベストアンサー選択の依頼

  • kei344

    2016/07/01 21:19

    コードはコードブロックで囲んでいただけませんか? ```(バッククオート3つ)で囲み、前後に改行をいれるか、コードを選択して「</>」ボタンを押すとコードブロックになります。また、HTMLも最低限追記ください。

    キャンセル

回答 2

checkベストアンサー

+2

<div class="top">
        <img src="img/imagebanner.png" width="1178" height="390" alt=""/>
        </div>
        <div class="digicatalog">
        <img src="img/dicatalogbtn.png" width="334" height="70" alt=""/>
        </div>      
</header>


まずこちらのHTML、</header>に対応する開始タグがありません。(多分<header class="header">が抜けてる?)

また、CSSの方で

.top {
    position:relative;
    width: 1181px;
    top: 0;
}

.digicatalog {
    position: absolute;
    width: 1181px;
    top: 255px;
    left: 820px;    
}


のように、全てのサイズでwidth:1181pxに固定されていますので、当然のことながら子要素の画像は伸縮しません。

あとコードの書き方として、メディアクエリでPCサイズ・タブレットサイズ・スマホサイズの3段階に完全に分断し、
それぞれの中で重複したコードを記述していますが、この書き方は無駄が多いのでオススメできません。

既存のサイトをレスポンシブ化しているとのことですので、既にPC向けのレイアウトを実現するためのCSSは存在していると思います。これをレスポンシブ化する際には、

/*ベースのCSS(PC向け・及び全サイズ共通の指定)*/

@media screen and (max-width: 960px) {
/*タブレットサイズ用の差分指定*/
}

@media screen and (max-width:640px){
/*スマホサイズ用の差分指定*/
}


のような形で、まずPC向けの基本スタイルを指定しておき、画面サイズが小さくなるにつれて必要な差分指定だけをメディアクエリで上書きする手法の方が一般的です。(デスクトップファースト方式)

参考までに。

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

+2

まずhtmlの方のimgタグについているwidth, heightの属性を削除しましょう。
これらの属性はレスポンシブデザインをする上では足を引っ張ります。

提示されてるサンプルコードだとこんな感じです。

<header><!-- 開始タグがなかったのでこの行は追記しました。 -->
    <div class="top">
        <img src="img/imagebanner.png" alt=""/>
    </div>
    <div class="digicatalog">
        <img src="img/dicatalogbtn.png" alt=""/>
    </div>
</header>

それからCSSの方で
各ブラウザサイズのimg向けにスタイル指定していきましょう。

/* PC向け */
@media screen and (min-width: 961px) {
    .top img {
        width: 1178px;
        height: 390px;
    }
    .digicatalog img {
        width: 334px;
        height: 70px;
    }
}

/* タブレット用 */
@media only screen and (min-width: 641px) and (max-width: 960px) {
    .top img {
        /* タブレット用のwidth, heightを入れて下さい */
    }
    .digicatalog img {
        /* タブレット用のwidth, heightを入れて下さい */
    }
}

/* スマホ用 */
@media screen and (max-width: 640px) {
    .top img {
        /* スマホ用のwidth, heightを入れて下さい */
    }
    .digicatalog img {
        /* スマホ用のwidth, heightを入れて下さい */
    }
}

これで画像の幅や高さは変更出来ると思います。
確認してみてください。

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2016/07/01 22:45

    初心者とのことなので
    老婆心ながらアドバイスさせて頂くと
    htmlのインデントを綺麗にする習慣をつけましょう。

    他の人がhtmlを読むときや、
    後で自分で書いたhtmlを見返す必要があるときに
    格段に見やすくなります。

    書いてすぐは構造を覚えていますが
    半年とか経つとすっかり忘れているので、
    後からメンテナンスやデザイン修正の依頼が来た時に
    対処しやすくなります。

    また、CSSに関しては
    aKusanoさんが書かれている回答のように
    既存のPC向けサイトをレスポンシブデザインにする場合は
    デスクトップファーストが効率的です。

    僕も2年前は質問者さんと同じぐらいのレベルだったので
    質問者さんもすぐわかるようになると思います。
    頑張ってください!

    キャンセル

15分調べてもわからないことは、teratailで質問しよう!

  • ただいまの回答率 89.21%
  • 質問をまとめることで、思考を整理して素早く解決
  • テンプレート機能で、簡単に質問をまとめられる