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

(当面の暫定で)ヘッダーに画像を含む文章を書く方法,画像をアップロードする場所などについてマニュアルに記載して欲しい #1131

Closed
1 task done
yegusa opened this issue Jan 25, 2016 · 9 comments

Comments

@yegusa
Copy link

yegusa commented Jan 25, 2016

(当面の暫定で)フッターに画像を含む文章を書く方法,画像をアップロードする場所などについてマニュアルに記載して欲しい #1118

のヘッダー版です。

@yegusa yegusa added this to the 1.2.0 milestone Jan 25, 2016
@yegusa
Copy link
Author

yegusa commented Jan 25, 2016

関連: ヘッダーも編集できるようにして欲しい #1112

@yegusa
Copy link
Author

yegusa commented Feb 1, 2016

たとえば、バナー画像: http://enju.next-l.jp/assets/logo.png をタイトルの代わりに張りたいとしたら、

https://raw.githubusercontent.com/next-l/enju_leaf/1.2/app/views/page/_header.html.erb

<h1 class="resource_title"><%= link_to LibraryGroup.system_name(@locale), root_path, title: LibraryGroup.system_name(@locale) %></h1>

↑をかえればよいのだとおもうのだけど、
どう変えればよいのか教えてくれるとマニュアルがかけそうです。

@yegusa
Copy link
Author

yegusa commented Feb 1, 2016

バナーの貼り方以外はマニュアルにかきました。
参照: http://next-l.github.io/manual/1.2/enju_install_vm_10.html#section10-6

@masao
Copy link
Member

masao commented Feb 28, 2016

先に質問のあった箇所は下記のような記述で書き換えられます:

      <h1><%= link_to image_tag('custom/logo.png'), root_path %></h1>

当該記述でロゴ画像に差し替えた例:
next l enju leaf

@masao masao added the checking label Feb 28, 2016
@scivola
Copy link
Contributor

scivola commented Feb 29, 2016

div#header_leftfloat: left になっているので,ある程度高さのある画像だとメニューに重なっちゃいますね。

@yegusa
Copy link
Author

yegusa commented Mar 26, 2016

現在のステータスメモ:

  • 確かめてクローズするだけ(江草でもできそう)

@yegusa
Copy link
Author

yegusa commented Mar 26, 2016

画像が、でかすぎるとはみ出してしまうんですね。
640x65px のサイズの画像だとちょうどよいようです。

確認しました。

image

vagrant@vagrant-ubuntu-trusty-64:~/enju$ bundle list | grep enju

  • enju_biblio (0.2.0.beta.1 f8ccc4c)
  • enju_circulation (0.2.0.beta.1 b8a143d)
  • enju_event (0.2.0.beta.1 ab338e3)
  • enju_leaf (1.2.0.beta.1 8243b36)
  • enju_library (0.2.0.beta.1 bd46d17)
  • enju_manifestation_viewer (0.2.0.beta.1 ce471f1)
  • enju_message (0.2.0.beta.1 afc79ce)
  • enju_ndl (0.2.0.beta.1 f142a0b)
  • enju_purchase_request (0.2.0.beta.1 9065073)
  • enju_subject (0.2.0.beta.1 5d33e0a)

@yegusa
Copy link
Author

yegusa commented Mar 26, 2016

まあ、とりあえず、 @scivola さんの指摘は、高さが小さ目の画像を作ってもらうことで回避するということで、いったんはクローズします。

マニュアルも書きました。

10-6 ヘッダーをカスタマイズする
http://next-l.github.io/manual/1.2/enju_install_vm_10.html#section10-6

@yegusa yegusa closed this as completed Mar 26, 2016
@scivola
Copy link
Contributor

scivola commented Mar 26, 2016

クローズされたので本来別 issue を立てるべきかもしれませんが,とりあえずここにメモします。

大きなヘッダー画像を置くとマズいのは,float: left だからというよりも,div#library_system_name と div#header_left のサイズががっつり固定されているからでした(下記参照)。

https://github.com/next-l/enju_leaf/blob/master/app/assets/stylesheets/enju.css#L55
https://github.com/next-l/enju_leaf/blob/master/app/assets/stylesheets/enju.css#L65

「天地 65px の画像がちょうどよい」というのは,div#header_left の height が 65px だからでしょう。

とりあえず,両者の height を auto にすればメニューと重ならなくなるようです。
(ここまで調べて力尽きました)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants