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

Organise boards selector: spacer between board families #64

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Organise boards selector: spacer between board families #64

wants to merge 1 commit into from

Conversation

jrial
Copy link

@jrial jrial commented Dec 1, 2018

This PR was not requested; feel free to suggest improvements, or to decline if you disagree with my changes. It was created because I always have trouble finding the right board in the long, flat and unsorted boards list. While I haven't implemented sorting (new to Coffeescript), I did add the following improvements which I find help me locate my boards quicker in the board selection:

  • Adds board family name, uppercased, between the different board families (Arduino, ESP32, ESP8266)
  • Adds a spacer between different board families to aid in locating the correct section
  • Renames '== Arduino Setting ==' to more descriptive 'Use Arduino IDE Setting'
  • Removes the redundant '== Custom Setting ==' entry.

Screenshots:

screenshot from 2018-12-01 18-53-42

screenshot from 2018-12-01 18-53-46

screenshot from 2018-12-01 18-53-52

* Adds board family name, uppercased, between the different board 
families (Arduino, ESP32, ESP8266)
* Adds a spacer between different board families to aid in locating the 
correct section
* Renames '== Arduino Setting ==' to more descriptive 'Use Arduino IDE 
Setting'
* Removes the redundant '== Custom Setting' entry.
@StaticRocket
Copy link
Collaborator

A nice idea, however I believe editing the menu's style could achieve a similar effect without allowing the user to select a null board.

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

2 participants