Omni-Notes
Omni-Notes copied to clipboard
Added an option for a 'minimal view' in addition to the reduced view …
…and extended view options.
Hi Nicholas,
great job!
Sorry for not having been completelly clear about any implementation's details into the issue #909 itself, I'll try to add more context here:
- Before start working on any modification could be useful to start a conversation abut that change to avoid pitfalls or working on something different of what is meant to be implemented
- Always give a look to the contributions guidelines (they're a little outdated, I'll give them a refresh as soon as possible, I promise) and be sure to test things if possible, show what the modifications apport to the app
In this specific case the requirement was to implement that view as alternative to the currently "reduced" one, avoiding adding another entry into the menu, here the description:
I'd really like to avoid overwhelming the main app's view with too many options, so this could eventually be an alternative to current Reduced view, to be choosed from inside settings. Exactly how that works right now color category colors
This because the future version 7.x will get a complete refresh of the UI, with card view, and no more than 3 view options should be available through menu into the notes list.
Another step further could be avoiding continue using "booleans and if/then" approach: if you want to try adding cleaner code you could try refactoring the whole thing in this sense (ex. creating an object that should represent the view mode and a factory to perform business logic on UI components depending on its state).
All of this just because when I started the app I didn't plannet to see it grow like this, so starting since a day I tried keeping code much cleaner for future management, and pull requests really have to do the same to help to continue like this.
Here's a little undesired effect: when you create a note with no title an empty card will be show