Consider renaming Heading "banner" modifier

DoSomething / forge

🎨 The DoSomething.org pattern library.

http://forge.dosomething.org

MIT License

48 stars 14 forks source link

Consider renaming Heading "banner" modifier #537

Closed DFurnes closed 8 years ago

DFurnes commented 8 years ago

Mentioned by @weerd in #536:

Does it make sense to still call this modifier .-banner? Just throwing that out there... I like the new look, but doesn't really seem like a banner now. Maybe .-emphasized

Since we don't want to break existing usage of this pattern, let's re-visit this when we make a major release sometime in the future.

DFurnes commented 8 years ago

Closed in #543! We'll remove the .heading.-banner entirely in Forge 7.x, whenever that happens.