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

会社情報ページ #45

Merged
merged 11 commits into from
Mar 24, 2018
Merged

会社情報ページ #45

merged 11 commits into from
Mar 24, 2018

Conversation

h-haraguchi
Copy link
Member

だいたいできた!レビューオナシャス。

いろいろな企業の「社長の言葉」ページを確認し、
それらの分析により抽出されたパターンを組み合わせて生成。
# Conflicts:
#	src/component/contents/TheGlobalHeader.vue
# Conflicts:
#	src/js/index.js
見出しの削除。
文章の微修正。
顔写真はダミーを入れとく。
@h-haraguchi h-haraguchi changed the title Issue 8 会社情報ページ Mar 19, 2018
@h-haraguchi h-haraguchi self-assigned this Mar 19, 2018
@h-haraguchi h-haraguchi mentioned this pull request Mar 19, 2018
2 tasks
Copy link
Member

@tronperidot tronperidot left a comment

Choose a reason for hiding this comment

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

質問の追加

<h2>社長の言葉</h2>
<p>
近年、iPhoneを始めとしたスマートフォンの普及とともに多くのイノベーションと人々の消費行動の変化が起きました。
また、スマートフォンでSNSを利用することで多くの人が情報の受信者であるだけでなく情報の発信者となりました。
Copy link
Member

Choose a reason for hiding this comment

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

pタグで改行を入れた時は半角の空白に変換された気がしましたけどどうでしたっけ。
(ソース上で改行入れないと見づらいと思うので、文章上で空白入っていても目立たなければこのままでも良いと思っています)

Copy link
Member Author

@h-haraguchi h-haraguchi Mar 19, 2018

Choose a reason for hiding this comment

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

ビルド時にスペース改行を消すオプション探したけど見つからなかったので諦めます!
vue-templateをvueify時にminify(preserveWhitespaceとかpreserveLineとか)するビルドオプションとかを探してみたんだけど残念ながら見つからんかったわ。昔はvueifyが依存してるhtml-minifierがどうにかしてくれてたらしいが、今は違うっぽい。
参照:

Copy link
Member

Choose a reason for hiding this comment

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

了解です!

Copy link
Member

@tronperidot tronperidot left a comment

Choose a reason for hiding this comment

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

スペースの問題はenv対応のときに一緒に探してみることにする。
なので、このプルリクはおk

@h-haraguchi h-haraguchi merged commit a2e110e into master Mar 24, 2018
@h-haraguchi h-haraguchi deleted the issue-8 branch March 24, 2018 04:42
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants