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

iOS で「会員登録をせずに購入」ボタンの文字が表示されないのを修正 #747

Merged
merged 1 commit into from
Jan 17, 2024

Conversation

nanasess
Copy link
Contributor

@nanasess nanasess commented Jul 23, 2023

iOS のデフォルトでは input[type=submit].color: #fff となっているため、添付画像のように 会員登録をせずに購入 ボタンが表示されない
image

input[type=submit].color: #000 とすることで対応

以下のPRを取り込むことでテストはすべて通ることを確認済
#767

@nanasess nanasess changed the title iOS で **会員登録をせずに購入** ボタンの文字が表示されないのを修正 iOS で「会員登録をせずに購入」ボタンの文字が表示されないのを修正 Jul 23, 2023
@codecov-commenter
Copy link

Codecov Report

Merging #747 (088b4c3) into master (47055de) will not change coverage.
The diff coverage is n/a.

❗ Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.

@@           Coverage Diff           @@
##           master     #747   +/-   ##
=======================================
  Coverage   55.33%   55.33%           
=======================================
  Files          75       75           
  Lines        8902     8902           
=======================================
  Hits         4926     4926           
  Misses       3976     3976           
Flag Coverage Δ
tests 55.33% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

@nanasess nanasess requested a review from nobuhiko August 2, 2023 15:33
@dotani1111 dotani1111 merged commit 0d32bb2 into EC-CUBE:master Jan 17, 2024
@nanasess nanasess added this to the 2.18(仮) milestone Dec 16, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants