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 a toggle to display menus wide #268

Merged
merged 1 commit into from Mar 29, 2020

Conversation

micoli
Copy link
Contributor

@micoli micoli commented Oct 6, 2019

A toggle is displayed at the upper right corner, when triggered
display menu are either displayed as dropdown menu, or like a
toolbar

Capture d’écran 2019-10-06 à 17 28 55

Capture d’écran 2019-10-06 à 17 28 45

@micoli micoli force-pushed the toggle-to-display-menu-wide branch from 37842d3 to b0e0ada Compare October 6, 2019 21:43
@codecov
Copy link

codecov bot commented Oct 6, 2019

Codecov Report

Merging #268 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #268   +/-   ##
=======================================
  Coverage   71.65%   71.65%           
=======================================
  Files           9        9           
  Lines         515      515           
  Branches      107      107           
=======================================
  Hits          369      369           
  Misses        146      146

Copy link
Contributor

@oktapodia oktapodia left a comment

Choose a reason for hiding this comment

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

Good work! I added a little comment

A toggle is displayed at the upper right corner, when triggered
display menu are eitehr displayed as dropdown menu, or like a
toolbar
@micoli micoli force-pushed the toggle-to-display-menu-wide branch from b0e0ada to a9b64a1 Compare February 8, 2020 05:37
@codecov-io
Copy link

Codecov Report

Merging #268 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #268   +/-   ##
=======================================
  Coverage   71.65%   71.65%           
=======================================
  Files           9        9           
  Lines         515      515           
  Branches      107      107           
=======================================
  Hits          369      369           
  Misses        146      146

@djfarrelly djfarrelly merged commit bc26793 into maildev:master Mar 29, 2020
@djfarrelly
Copy link
Member

After adding the new toggle design in #291, I moved the setting toggle there which now also supports persisting in local storage

Image 2020-03-29 at 2 54 33 PM

@djfarrelly djfarrelly added this to the 2.0.0 milestone Mar 30, 2020
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.

None yet

4 participants