Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

新規お届け先を追加画面のMobile表示で変更ボタンが表示されていない #2044

Merged
merged 5 commits into from
Feb 28, 2017

Conversation

hoand-vn
Copy link
Contributor

#1941 : Adjust position button 'change' address in my page

@Yangsin Yangsin added Status: needs-review bug:Middle affected:template フロントテンプレートの変更 labels Jan 22, 2017
@Yangsin Yangsin added this to the 3.0.14 milestone Jan 22, 2017
@ryo-endo
Copy link
Contributor

末尾に追加するのではなく、既存cssの適切な位置に挿入してください。
よろしくお願いします。

@lammn
Copy link
Contributor

lammn commented Jan 25, 2017

@ryo-endo i fixed please confirm

@t-nagahashi
Copy link
Contributor

スマホでの動作確認も問題ありませんでした。

@ryo-endo
Copy link
Contributor

ryo-endo commented Feb 6, 2017

@chihiro-adachi @t-nagahashi
ピクセル数決めうちでボタンの位置を指定しているのは、微妙じゃないでしょうか?
個人的にはposition: relative;だけでいいんじゃないかと。

@ryo-endo
Copy link
Contributor

ryo-endo commented Feb 6, 2017

position: relativeを指定した時の挙動が、ブラウザによって異なるため、
一通りのブラウザ環境で確認すること。

@t-nagahashi
Copy link
Contributor

できれば、真ん中寄せにしたいですね。
難しいようであれば、ピクセル指定の対応のみやめる形にしましょう。

Copy link
Contributor

@t-nagahashi t-nagahashi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

sourece code isOK.

@t-nagahashi
Copy link
Contributor

動作確認もPC、スマホともにOK。
(住所も最大文字数の32文字表示でも、変更ボタンが真ん中に表示されることを確認。

@@ -3251,8 +3254,10 @@ fieldset[disabled] .btn-link:focus {
transform: translateX(0);
}



/****** MyPage Add address ********/
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ここだけコメントの形式が異なるようです。

@@ -4330,6 +4335,11 @@ fieldset[disabled] .btn-link:focus {
padding-top: 8px;
}

/****** MyPage Add address ********/
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

こちらもコメント形式が他と異なります。

@t-nagahashi
Copy link
Contributor

ソースコードのコメントが修正されていることが確認できたため、OKとします。

@ryo-endo ryo-endo merged commit 9a70e6e into EC-CUBE:master Feb 28, 2017
@t-nagahashi t-nagahashi changed the title Fix display in smart phone 新規お届け先を追加画面のMobile表示で変更ボタンが表示されていない Mar 1, 2017
@lqdung-lockon lqdung-lockon deleted the bug_fix_#1941 branch September 26, 2017 07:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
affected:template フロントテンプレートの変更 bug:Middle Status: ready-for-merge
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

7 participants