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

Add documentation about title slide meta tags #165

Conversation

makicamel
Copy link
Contributor

I add documentation about title slide meta tags.

Copy link
Member

@kou kou left a comment

Choose a reason for hiding this comment

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

Thanks!

Comment on lines 62 to 64
specified when starting Rabbit. If both (({start_time}))
and (({end_time})) are specified, no need to specify
(({allotted_time})) as it will be calculated from those
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
specified when starting Rabbit. If both (({start_time}))
and (({end_time})) are specified, no need to specify
(({allotted_time})) as it will be calculated from those
specified when starting Rabbit. If both `start_time`
and `end_time` are specified, no need to specify
`allotted_time` as it will be calculated from those

Comment on lines 65 to 66
使用されます。(({start_time}))と(({end_time}))を両方指定している
場合はそれらの属性から計算されるため、(({allotted_time}))を指定する
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
使用されます。(({start_time}))と(({end_time}))を両方指定している
場合はそれらの属性から計算されるため、(({allotted_time}))を指定する
使用されます。`start_time``end_time`を両方指定している
場合はそれらの属性から計算されるため、`allotted_time`を指定する

@makicamel makicamel force-pushed the add-documentation-about-title-slide-meta-tags branch from 85b6aa3 to 876c1ce Compare September 15, 2024 11:32
@makicamel
Copy link
Contributor Author

Wops ... I fixed 🙏

@kou kou merged commit 95eff38 into rabbit-shocker:main Sep 15, 2024
3 checks passed
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.

2 participants