View Issue Details

IDProjectCategoryView StatusLast Update
0003235ardourfeaturespublic2020-04-19 20:14
Reporterdanboid Assigned Topaul  
PrioritynormalSeverityfeatureReproducibilityN/A
Status closedResolutionfixed 
Summary0003235: mouse over MIDI note should display note velocity value
DescriptionUnder 7253 you have to change the velocity of a note to see its exact value which is awkward- it would be much more convenient if it was displayed along with the note value when you mouse over a note.
TagsNo tags attached.

Activities

cth103

2010-06-11 21:27

administrator   ~0008223

Any suggestions on how the mouse-over text should be formatted?

danboid

2010-06-11 21:44

reporter   ~0008224

Hi Carl

Prob the most easy and logical way to implement it, if not necessarily the best, is simply to reveal the same text that you see when adjusting note velocity. This text is very small though and you have to be zoomed in quite close to be able to read it.

I'm typing this on a machine that doesn't have A3 installed but iirc when you mouse over a note you get that notes pitch value displayed in easily readable text just above the cursor. If the note value gets displayed just above it would be nice to have the notes velocity displayed just below the cursor in the same size font, maybe prefixed by 'Vel:' to make it clear what it represents.

2010-06-13 11:06

 

3235.patch (1,017 bytes)   
diff --git a/gtk2_ardour/midi_region_view.cc b/gtk2_ardour/midi_region_view.cc
index cbd89dd..2dad271 100644
--- a/gtk2_ardour/midi_region_view.cc
+++ b/gtk2_ardour/midi_region_view.cc
@@ -2541,6 +2541,7 @@ MidiRegionView::note_entered(ArdourCanvas::CanvasNoteEvent* ev)
 	}
 
 	show_verbose_canvas_cursor (ev->note ());
+	ev->show_velocity ();
 }
 
 void
@@ -2551,6 +2552,7 @@ MidiRegionView::note_left (ArdourCanvas::CanvasNoteEvent* note)
 	}
 
 	trackview.editor().hide_verbose_canvas_cursor ();
+	note->hide_velocity ();
 }
 
 void
@@ -2869,8 +2871,8 @@ MidiRegionView::snap_changed ()
 void
 MidiRegionView::show_verbose_canvas_cursor (boost::shared_ptr<NoteType> n) const
 {
-	char buf[12];
-	snprintf (buf, sizeof (buf), "%s (%d)", Evoral::midi_note_name (n->note()).c_str(), (int) n->note ());
+	char buf[32];
+	snprintf (buf, sizeof (buf), "%s (%d) vel %d", Evoral::midi_note_name (n->note()).c_str(), (int) n->note (), (int) n->velocity());
 	trackview.editor().show_verbose_canvas_cursor_with (buf);
 }
 
3235.patch (1,017 bytes)   

cth103

2010-06-13 11:06

administrator   ~0008226

The attached patch does both. Any thoughts?

danboid

2010-06-13 18:18

reporter   ~0008227

Last edited: 2010-06-13 18:24

With that patch you can can see the note vel on a note on mouse over in little text on top of the note as well as in large text to the side with the notes pitch BUT you can only see the little text on the note for non-selected notes.

I was going to suggest dropping the little text anyway but seeing as we we have this potential buglet now its given me another reason to suggest scrapping the tiny text for just displaying the notes velocity alongside its pitch when you mouse over. This would of course mean that this text would need to be displayed when modifying a notes velocity as the big text disappears atm when doing that.

Thanks!

danboid

2010-06-15 20:08

reporter   ~0008248

There is one situation where the tiny text makes sense and thats when you are adjusting the velocity of multiple notes each with a unique velocity value. In this situation you wouldn't really be able to display the larger text as well as the pitch/ note value info too without lots of overlap and clutter in many cases.

danboid

2010-06-18 15:51

reporter   ~0008269

Last edited: 2010-06-18 15:52

Working for me in 7279 - well except for the bug I posted earlier

Thanks CH!

system

2020-04-19 20:14

developer   ~0022127

Issue has been closed automatically, by Trigger Close Plugin.
Feel free to re-open with additional information if you think the issue is not resolved.

Issue History

Date Modified Username Field Change
2010-06-11 12:38 danboid New Issue
2010-06-11 21:27 cth103 Note Added: 0008223
2010-06-11 21:27 cth103 Status new => feedback
2010-06-11 21:44 danboid Note Added: 0008224
2010-06-13 11:06 cth103 File Added: 3235.patch
2010-06-13 11:06 cth103 Note Added: 0008226
2010-06-13 18:18 danboid Note Added: 0008227
2010-06-13 18:21 danboid Note Edited: 0008227
2010-06-13 18:21 danboid Note Edited: 0008227
2010-06-13 18:24 danboid Note Edited: 0008227
2010-06-15 20:08 danboid Note Added: 0008248
2010-06-18 15:51 danboid Note Added: 0008269
2010-06-18 15:52 danboid Note Edited: 0008269
2010-06-29 13:23 paul cost => 0.00
2010-06-29 13:23 paul Status feedback => resolved
2010-06-29 13:23 paul Resolution open => fixed
2010-06-29 13:23 paul Assigned To => paul
2020-04-19 20:14 system Note Added: 0022127
2020-04-19 20:14 system Status resolved => closed