Skip to content

Conversation

@AWolf81
Copy link
Contributor

@AWolf81 AWolf81 commented Feb 15, 2020

Description

I'm adding the requested changes for PR #3456 - @MiloTodt I hope that's OK.
Changed to the new apply-theme introduced with the Nord theme.

It would be great if we could change the loop inside NoteList.styl to only use $themes var. But adding the other names to the variable in index.styl was not working.
Created an issue in NoteItemSimple.styl, so I've kept the two loops.

Issue fixed

#3455

Type of changes

  • ⚪ Bug fix (Change that fixed an issue)
  • ⚪ Breaking change (Change that can cause existing functionality to change)
  • 🔘 Improvement (Change that improves the code. Maybe performance or development improvement)
  • ⚪ Feature (Change that adds new functionality)
  • ⚪ Documentation change (Change that modifies documentation. Maybe typo fixes)

Checklist:

  • 🔘 My code follows the project code style
  • ⚪ I have written test for my code and it has been tested
  • ⚪ All existing tests have been passed
  • ⚪ I have attached a screenshot/video to visualize my change if possible

Milo Todt and others added 5 commits February 1, 2020 16:00
* added rtl toggle button

* added rtl toggle button

* keep code styling aligned to the left and ltr at all times

* added hotkey setting for direction toggle

* fixed requested changes

* fix undefined variable

* Copyright info update

updated Copyright (C) 2017 - 2019 BoostIO to Copyright (C) 2017 - 2020 BoostIO

* Refine Chinese translation

- Should not translate "space", it means space key in most context
- Should translate "keymap", "spellcheck disabled", "auto detect"
- Should translate "On Right Click" to "右键点击"
- Refine misc translation.

* Added Wiki Link

* Add Traditional Chinese option to build.md

* Bug fix (sets tabWith to 2 on prettier configuration and ConfigManager so checkboxes can be clikable)

Co-authored-by: ibraude <[email protected]>
Co-authored-by: Junyoung Choi <[email protected]>
Co-authored-by: Satyendra <[email protected]>
Co-authored-by: Andrew <[email protected]>
@ZeroX-DG ZeroX-DG added the awaiting review ❇️ Pull request is awaiting a review. label Feb 16, 2020
Copy link
Member

@ZeroX-DG ZeroX-DG left a comment

Choose a reason for hiding this comment

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

LGTM 🎉 thank you:)

@ZeroX-DG ZeroX-DG added approved 👍 Pull request has been approved by sufficient reviewers. and removed awaiting review ❇️ Pull request is awaiting a review. labels Feb 20, 2020
@Rokt33r Rokt33r added this to the v0.15.0 milestone Feb 24, 2020
@Rokt33r Rokt33r merged commit feb2a87 into BoostIO:master Feb 24, 2020
@AWolf81 AWolf81 deleted the 3455_dropdown-colour branch February 24, 2020 10:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

approved 👍 Pull request has been approved by sufficient reviewers.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants