Skip to content

Change row color in session View #enhancement#163

Merged
jamiefaye merged 7 commits into
SynthstromAudible:communityfrom
kleinstar1:community
Jul 12, 2023
Merged

Change row color in session View #enhancement#163
jamiefaye merged 7 commits into
SynthstromAudible:communityfrom
kleinstar1:community

Conversation

@kleinstar1
Copy link
Copy Markdown
Contributor

Change row color by pressing row & shift & vertical encoder. This is the same shortcut as in clip view.
This helps a lot to setup clip colors - it shows the resulting line colors right in place in song/session view.

Change row color by pressing row & shift & vertical encoder. This is the same shortcut as in clip view.
@kleinstar1 kleinstar1 changed the title Change row color in session View Change row color in session View #enhancement Jul 6, 2023
@sapphire-arches
Copy link
Copy Markdown
Collaborator

Thanks for the contribution! If you don't mind, can you add some documentation for the user interface for this feature to the community features document?

Otherwise I think this works well. CI failures are spurious and it works locally on my OLED deluge.

@kleinstar1
Copy link
Copy Markdown
Contributor Author

kleinstar1 commented Jul 7, 2023

Oh, I missed that - was wondering where to put that and where to find the new features list :)
I hope that this is now included in the pull request. I could not manage to set the link in the document to PR #163 correctly :(

@sapphire-arches
Copy link
Copy Markdown
Collaborator

The trick is you need to add it to the list at the end of the file. See 479a27b for an example.

Copy link
Copy Markdown
Collaborator

@sapphire-arches sapphire-arches left a comment

Choose a reason for hiding this comment

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

Some editorial comments as long as you're going to update this file again. Thanks again for the contribution!

Comment thread CommunityFeatures.md Outdated

### Instrument Keyboard View
- ([#46]) Note offset between rows is now configurable by holding shift and using the horizontal encoder. This allows e.g. an isomorphic keyboard layout by setting the row offset to 12. The setting is saved per clip in the song file.
### Song View Row Colors
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

(minor nitpicks sorry, but I figure you're going to be in the area anyway...)

Should remove "Row Colors" here, since this section will have all the changes for "Song View"

Comment thread CommunityFeatures.md Outdated
### Instrument Keyboard View
- ([#46]) Note offset between rows is now configurable by holding shift and using the horizontal encoder. This allows e.g. an isomorphic keyboard layout by setting the row offset to 12. The setting is saved per clip in the song file.
### Song View Row Colors
- ([#163]) Pressing a clip row + shift & scorll vertically changes the selected row color. This is the same shortcut like before when setting the color in the clip view.
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Spelling error, "scorll" -> "scroll"

Comment thread CommunityFeatures.md
## New behaviors

### Instrument Keyboard View
- ([#46]) Note offset between rows is now configurable by holding shift and using the horizontal encoder. This allows e.g. an isomorphic keyboard layout by setting the row offset to 12. The setting is saved per clip in the song file.
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Probably shouldn't delete this section =)

@kleinstar1
Copy link
Copy Markdown
Contributor Author

Hope is better now - sorry for beeing beginnerisch :)
Thanks for checking this out.
Next week I`ll build the community branch and put into the forum together with the documentation for all the none-programmers out there.
Or did I miss releases somewhere?

@jamiefaye
Copy link
Copy Markdown
Collaborator

Looking forward to adding this one. Let me know when its ready to go.

@sapphire-arches
Copy link
Copy Markdown
Collaborator

Hope is better now - sorry for beeing beginnerisch :) Thanks for checking this out. Next week I`ll build the community branch and put into the forum together with the documentation for all the none-programmers out there. Or did I miss releases somewhere?

Mostly LGTM, just 1 space missing that's upsetting the format checker.

We sort of intentionally don't have a release yet, as we're not yet confident in the stability of the firmware. ok-reza is working on a community update document which I think he's planning to post to the forum and the facebook group in the next couple of days. If you want to brag about your features or ask for alpha testing don't let that stop you though.

@sichtbeton
Copy link
Copy Markdown
Contributor

sichtbeton commented Jul 8, 2023

Would it be possible to add the ability to color a row with one constant value, making the row opaque and the session view much more readable?

@sapphire-arches
Copy link
Copy Markdown
Collaborator

sapphire-arches commented Jul 8, 2023 via email

@kleinstar1
Copy link
Copy Markdown
Contributor Author

Would it be possible to add the ability to color a row with one constant value, making the row opaque and the session view much more readable?

This would throw away all current color calculations and thoughts from the synthstrom developers. The community somehow has to say what should be the best. This change would also be too much for my skills with this and code knowledge so far :)
I think the ability to change the row color in song view is good for now and helps a lot.
In the future I would love to see the zoom level for the song seperated from the clip view because this changes the colors as well.

@kleinstar1
Copy link
Copy Markdown
Contributor Author

Looking forward to adding this one. Let me know when its ready to go.

I hopefully fixed the spell checking and I hope that is ready to go :)

@kleinstar1
Copy link
Copy Markdown
Contributor Author

Hope is better now - sorry for beeing beginnerisch :) Thanks for checking this out. Next week I`ll build the community branch and put into the forum together with the documentation for all the none-programmers out there. Or did I miss releases somewhere?

Mostly LGTM, just 1 space missing that's upsetting the format checker.

We sort of intentionally don't have a release yet, as we're not yet confident in the stability of the firmware. ok-reza is working on a community update document which I think he's planning to post to the forum and the facebook group in the next couple of days. If you want to brag about your features or ask for alpha testing don't let that stop you though.

I saw that recently :)
It`s very nice to see this all happen :)

@kleinstar1
Copy link
Copy Markdown
Contributor Author

Hey,
could someone please help me with merging?
I think it`s not possible due to the checks.. but where can I start them again and where do I see the mistakes?
Thank you

@sapphire-arches
Copy link
Copy Markdown
Collaborator

A maintainer needs to approve the checks so they run. I think this is ready to merge otherwise, sorry for the wait!

@litui litui added the enhancement New feature or request label Jul 12, 2023
@jamiefaye jamiefaye added this pull request to the merge queue Jul 12, 2023
Merged via the queue into SynthstromAudible:community with commit 28d69cb Jul 12, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants