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

CSS: Don't override font-weight for section (rebased on dev) #691

Merged
merged 1 commit into from
Nov 7, 2013

Conversation

fghaas
Copy link
Contributor

@fghaas fghaas commented Nov 7, 2013

.slides>section and .slides>section>section set the font-weight to
"normal". This overrides any font-weight setting a theme may set at
the body or .reveal level, and requires a theme author to also
specifically set the font-weight for .slides>section and
.slides>section>section. That's tedious and also counterintuitive.

Adopt a saner default by setting the font-weight to inherit.

.slides>section and .slides>section>section set the font-weight to
"normal". This overrides any font-weight setting a theme may set at
the body or .reveal level, and requires a theme author to also
specifically set the font-weight for .slides>section and
.slides>section>section. That's tedious and also counterintuitive.

Adopt a saner default by setting the font-weight to inherit.
hakimel added a commit that referenced this pull request Nov 7, 2013
CSS: Don't override font-weight for section
@hakimel hakimel merged commit 3a8bcb1 into hakimel:dev Nov 7, 2013
@hakimel
Copy link
Owner

hakimel commented Nov 7, 2013

Great, thanks for that!

hakimel added a commit that referenced this pull request Nov 8, 2013
hakimel added a commit that referenced this pull request Nov 8, 2013
pulni pushed a commit to pulni/reveal.js that referenced this pull request Nov 18, 2013
* upstream/dev: (92 commits)
  slide number changes; config value renamed, added to readme, moved to right side of screen
  make background transition default to exact same speed as normal transitions
  change transition easing for zooms
  rebuild css with change from hakimel#691
  define normal font widths in all themes to make sure things look the same after hakimel#691
  CSS: Don't override font-weight for section
  remove old inaccurate comment
  comments
  data-autoslide on vertical parent now applies to all child slides hakimel#685
  ensure that all fragments on past slides are visible hakimel#684
  add code slide to markdown example hakimel#682
  readme adjustment
  Add slide attributes for single slide section case.
  Add mardown-based slide attribute examples.
  Add unit test for default slide attributes separator pattern.
  Remove console.log debugs, and fix tabs
  Fix default separator pattern for attribute in slide.
  Restore default notation for attributes in element.
  Restore proper pattern for attributes in elements.
  Add QUnit.js unit tests for attributes in Markdown slides.
  ...
m4dz pushed a commit to m4dz/reveal.js that referenced this pull request Apr 16, 2014
enovajuan pushed a commit to enovajuan/reveal.js that referenced this pull request Dec 2, 2014
CSS: Don't override font-weight for section
enovajuan pushed a commit to enovajuan/reveal.js that referenced this pull request Dec 2, 2014
enovajuan pushed a commit to enovajuan/reveal.js that referenced this pull request Dec 2, 2014
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