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.

Sign up for GitHub

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

Jump to bottom

Add the option to hide the first column of the interval matrix #445

Open
frostburn opened this issue Jun 18, 2023 · 0 comments
Open

Add the option to hide the first column of the interval matrix #445

frostburn opened this issue Jun 18, 2023 · 0 comments
Milestone
User requests

Comments

@frostburn
Copy link
Member

inthar — 06/16/2023 4:37 PM

I don't know if I've already raised this issue, but the first column of the interval matrix is wholly unnecessary.

image
@frostburn frostburn added this to the User requests milestone Jun 18, 2023
@frostburn frostburn linked a pull request Nov 27, 2023 that will close this issue
Add a toggle for interval matrix indexing (Issue #411) #500
Closed
inthar-raven added a commit to inthar-raven/scale-workshop that referenced this issue Nov 28, 2023
@inthar-raven
Update AnalysisView.vue
d07aa9c 
The table now keeps the 0-step column for both 0-indexed and 1-indexed, so we can now address xenharmonic-devs#445 if we want.
inthar-raven added a commit to inthar-raven/scale-workshop that referenced this issue Nov 29, 2023
@inthar-raven
Update analysis.ts
27b759d 
startIndex is not necessary yet (issue xenharmonic-devs#445 should be dealt with separately)
inthar-raven added a commit to inthar-raven/scale-workshop that referenced this issue Nov 29, 2023
@inthar-raven
Issue xenharmonic-devs#411, add radio buttons to change the indexing …
995ab97 
…of the top row of the interval matrix in Analysis view.

Add files via upload

Update AnalysisView.vue

The table now keeps the 0-step column for both 0-indexed and 1-indexed, so we can now address xenharmonic-devs#445 if we want.

Update AnalysisView.vue

`indexing` -> `intervalMatrixIndexing`

Update App.vue

Persist `intervalMatrixIndex` in local storage

Update AnalysisView.vue

emit value `intervalMatrixIndex` to persist it

Add files via upload

Add files via upload

Add files via upload

`intervalMatrixIndexing` is now a `number`.

`intervalMatrixIndexing` is now a `number`.

Update AnalysisView.vue

`value="0"` etc. -> `:value="0"` etc.

Update AnalysisView.vue

Update AnalysisView.vue

Make sure to parse the integer emitted by indexing radio buttons as decimal

Update AnalysisView.vue

Update analysis.ts

startIndex is not necessary yet (issue xenharmonic-devs#445 should be dealt with separately)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
User requests
Development

Successfully merging a pull request may close this issue.

Add a toggle for interval matrix indexing (Issue #411)
1 participant
@frostburn