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.

Sign up for GitHub

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

Jump to bottom

<section>のサポート #1685

Merged
merged 3 commits into from
Jun 25, 2021
Merged

<section>のサポート #1685

merged 3 commits into from
Jun 25, 2021

Conversation

kmuto
Copy link
Owner

@kmuto kmuto commented Apr 10, 2021

CSS組版との親和性を上げるため、
HTMLビルダにおいて、headlineレベルに基づき <section> 要素で囲んで構造化する機能を追加しようと思います。通常の見出しのみが対象で、コラムについてはこれまでどおりdivです。

Makerに応じて epubmaker/use_section: true か webmaker/use_section: true で有効になります。

@takahashim
Copy link
Collaborator

いろいろ気になるのですが、この <section> の使い方はCSS組版の仕様（？）か何かで決まっているんでしょうか…？（特に見出しレベルが飛んだ時に多重のネストになる挙動）

@kmuto
Copy link
Owner Author

kmuto commented Apr 11, 2021

なるほど、たぶんこうだろうという推測だけで実装していましたが、Vivliostyleでmdから変換してみると違いました…。スキップすることはなく、より上位のものがきたらsectionを閉じる、ということでしたね。

# H1

##### H5

## H2

#### H4

### H3

#### H4-2

## H2-2

  <body>
    <section id="h1">
      <h1>H1</h1>
      <section id="h5">
        <h5>H5</h5>
      </section>
      <section id="h2">
        <h2>H2</h2>
        <section id="h4">
          <h4>H4</h4>
        </section>
        <section id="h3">
          <h3>H3</h3>
          <section id="h4-2">
            <h4>H4-2</h4>
          </section>
        </section>
      </section>
      <section id="h2-2">
        <h2>H2-2</h2>
      </section>
    </section>
  </body>

いただいたURLも見ながら考えます。

@kmuto kmuto changed the title <section>のサポート [WIP] <section>のサポート Apr 12, 2021
@kmuto kmuto changed the title [WIP] <section>のサポート <section>のサポート Apr 25, 2021
@kmuto
Copy link
Owner Author

kmuto commented Apr 25, 2021

下位sectionを開くときも1段階のみ開くように修正しました。

<section> の考え方はh1を入れたり入れなかったりとあるみたいで統一見解がないのですが、Vivliostyleの変換物のほうではh1も囲んでいたのでそちらに合わせています(大きめな目的としてVivliostyleが現在進めているCSSテーマを流用できればというのが理由なので)。

@takahashim
Copy link
Collaborator

改めて確認しました(VFM・Pandocも)。
いろいろ気になる点はあるのですが、方向性は理解したのでマージしてから別途PRします！

@takahashim takahashim merged commit 5ec2e0c into master Jun 25, 2021
@takahashim takahashim deleted the html-section branch June 25, 2021 15:22
kmuto added a commit that referenced this pull request Jun 27, 2021
@kmuto
#1685
e6b8253
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@kmuto @takahashim