Skip to content

Conversation

@Martii
Copy link
Member

@Martii Martii commented Feb 6, 2015

  • Don't affect non-small alerts
  • Assuming our main nav will always be at top... snug up the margin-bottom
  • Add class to reminders
  • Snug up reminders
  • Undo default alert padding change and *(x) close from Smaller alerts #568 and handle only in reminders that are small
  • Use some more LESS syntax in staged area
  • Add _bootstrap_s default fade effect on close... the timing is a bit short so not really noticable but if you stare it closely it does use CSS3.

NOTES:

  • When removing reminders, which are currently static in the view, leave the parent reminders div in to maintain current spacing especially when closed.

* Don't affect non-small alerts
* Assuming our main nav will always be at top... snug up the `margin-bottom`
* Add class to reminders
* Snug up reminders
* Undo default alert padding change and *(x) close from OpenUserJS#568 and handle only in reminders that are small
* Use some more LESS syntax in staged area
* Add *bootstrap*s default fade effect on close... the timing is a bit short so not really noticable but if you stare it closely it does use CSS3.

**NOTES**:
* When removing reminders, which are currently static in the view, leave the parent `reminders` div in to maintain current spacing especially when closed.
Martii added a commit that referenced this pull request Feb 6, 2015
Snug up top reminders in CSS

Auto-merge
@Martii Martii merged commit 5c3d597 into OpenUserJS:master Feb 6, 2015
@Martii Martii added UI Pertains inclusively to the User Interface. enhancement Something we do have implemented already but needs improvement upon to the best of knowledge. labels Feb 6, 2015
@Martii Martii deleted the snugUpReminders branch February 6, 2015 22:41
@github-actions github-actions bot locked as resolved and limited conversation to collaborators May 20, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

enhancement Something we do have implemented already but needs improvement upon to the best of knowledge. UI Pertains inclusively to the User Interface.

Development

Successfully merging this pull request may close these issues.

1 participant