More condensed tabular code formatting in selected areas

[eugenesvk]

While reading the auto-cargo-formatted code I sometimes get lost in the copious amounts of whitespace and misalignments obscuring patters, so to make it readable I sometimes reformat to a simple table where it's easy to skip the whole "column" of identical "returns" and only focus on the difference

For example, here it's clear that:

• there is a bunch of identical literal matches, each returning some action
• most of these are custom actions, but it's easy to see which aren't since action type is in the same column
• (and if not for the very long names or with a full screen view) you can even see what the actions are

and if you need more details, you could turn wrapping off and stil maintain most of the alignment

The default formatting is all zigzaggy, so harder to see the difference (in one spot sref/sref/sref_slice is one one line, in another it's spread over two lines, so harder to see that's it's all identical sequence of commands), besides taking more space for empty } and )))

Now, I'm not suggesting to reformat everything, just wondering whether you'd be open to accepting a few slaps of #[rustfmt::skip] here and there to get to the more aligned output

(or maybe there are some formatting opts that can achieve the same thing)

[jtroo]

Hmm looking at this code again, perhaps instead of playing with/bypassing rustfmt, the solution to detail-finding woes is to create a helper fn to do all the boilerplate of putting the custom action into the right type.

[jtroo]

Seems to be an improvement: 426a0c0

[eugenesvk]

Thanks, it is an improvement, though a table is still better :)

match ac {
  "_"|"‗"|"≝"                	=> return Ok(s.a.sref(Action::Trans )),
  "XX"|"✗"|"∅"|"•"           	=> return Ok(s.a.sref(Action::NoOp  )),
  "rpt-any"                  	=> return Ok(s.a.    (Action::Repeat)),
  "sldr"                     	=>{return custom(CustomAction::SequenceLeader	(s.default_sequence_timeout,s.default_sequence_input_mode,),&s.a,)}
  "scnl"                     	=> return custom(CustomAction::SequenceCancel	                     	 ,&s.a),
  "lrld"                     	=> return custom(CustomAction::LiveReload    	                     	 ,&s.a),
  "lrld-next"|"lrnx"         	=> return custom(CustomAction::LiveReloadNext	                     	 ,&s.a),
  "lrld-prev"|"lrpv"         	=> return custom(CustomAction::LiveReloadPrev	                     	 ,&s.a),
  "mlft"|"mouseleft"         	=> return custom(CustomAction::Mouse         	(Btn::Left           	),&s.a),
  "mrgt"|"mouseright"        	=> return custom(CustomAction::Mouse         	(Btn::Right          	),&s.a),
  "mmid"|"mousemid"          	=> return custom(CustomAction::Mouse         	(Btn::Mid            	),&s.a),
  "mfwd"|"mouseforward"      	=> return custom(CustomAction::Mouse         	(Btn::Forward        	),&s.a),
  "mbck"|"mousebackward"     	=> return custom(CustomAction::Mouse         	(Btn::Backward       	),&s.a),
  "mltp"|"mousetapleft"      	=> return custom(CustomAction::MouseTap      	(Btn::Left           	),&s.a),
  "mrtp"|"mousetapright"     	=> return custom(CustomAction::MouseTap      	(Btn::Right          	),&s.a),
  "mmtp"|"mousetapmid"       	=> return custom(CustomAction::MouseTap      	(Btn::Mid            	),&s.a),
  "mftp"|"mousetapforward"   	=> return custom(CustomAction::MouseTap      	(Btn::Forward        	),&s.a),
  "mbtp"|"mousetapbackward"  	=> return custom(CustomAction::MouseTap      	(Btn::Backward       	),&s.a),
  "mwu" |"mousewheelup"      	=>{return custom(CustomAction::MWheelNotch   	{dir:MWheelDir::Up   	},&s.a,)}
  "mwd" |"mousewheeldown"    	=>{return custom(CustomAction::MWheelNotch   	{dir:MWheelDir::Down 	},&s.a,)}
  "mwl" |"mousewheelleft"    	=>{return custom(CustomAction::MWheelNotch   	{dir:MWheelDir::Left 	},&s.a,)}
  "mwr" |"mousewheelright"   	=>{return custom(CustomAction::MWheelNotch   	{dir:MWheelDir::Right	},&s.a,)}
  "rpt" |"repeat"|"rpt-key"  	=> return custom(CustomAction::Repeat        	                     	 ,&s.a),
  "dynamic-macro-record-stop"	=>{return custom(CustomAction::DynamicMacroRecordStop(0), &s.a)}
  _                          	=>{}
}

[jtroo]

I would say the commit is an objective improvement, while the table is subjective.

For example, too much whitespace makes this harder to read for me:

  "scnl"                     	=> return custom(CustomAction::SequenceCancel	                     	 ,&s.a),
  "lrld"                     	=> return custom(CustomAction::LiveReload    	                     	 ,&s.a),

And then every line also becomes too wide compared to most other lines, since I have my terminal windows set to near 100 columns. The text wrapping is not pleasant to my eyes.

[eugenesvk]

The ease of tracking differences is an objective benefit of alignment, the downside of too much spacing could be partially addressed by trading off some (but not all like the usual formatting does) of that benefit with smaller groups

match ac {
    "_"|"‗"|"≝"     	=> return Ok(s.a.sref(Action::Trans )),
    "XX"|"✗"|"∅"|"•"	=> return Ok(s.a.sref(Action::NoOp  )),
    "rpt-any"       	=> return Ok(s.a.sref(Action::Repeat)),
    //Short
    "sldr"	=>{return custom(CustomAction::SequenceLeader	(s.default_sequence_timeout,s.default_sequence_input_mode,),&s.a,)},
    "scnl"	=> return custom(CustomAction::SequenceCancel	 ,&s.a),
    "lrld"	=> return custom(CustomAction::LiveReload    	 ,&s.a),
    //Longer
    "lrld-next"|"lrnx"	=> return custom(CustomAction::LiveReloadNext	 ,&s.a),
    "lrld-prev"|"lrpv"	=> return custom(CustomAction::LiveReloadPrev	 ,&s.a),
    //Mouse
    "mlft"|"mouseleft"       	=> return custom(CustomAction::Mouse   (Btn::Left    ),&s.a),
    "mrgt"|"mouseright"      	=> return custom(CustomAction::Mouse   (Btn::Right   ),&s.a),
    "mmid"|"mousemid"        	=> return custom(CustomAction::Mouse   (Btn::Mid     ),&s.a),
    "mfwd"|"mouseforward"    	=> return custom(CustomAction::Mouse   (Btn::Forward ),&s.a),
    "mbck"|"mousebackward"   	=> return custom(CustomAction::Mouse   (Btn::Backward),&s.a),
    "mltp"|"mousetapleft"    	=> return custom(CustomAction::MouseTap(Btn::Left    ),&s.a),
    "mrtp"|"mousetapright"   	=> return custom(CustomAction::MouseTap(Btn::Right   ),&s.a),
    "mmtp"|"mousetapmid"     	=> return custom(CustomAction::MouseTap(Btn::Mid     ),&s.a),
    "mftp"|"mousetapforward" 	=> return custom(CustomAction::MouseTap(Btn::Forward ),&s.a),
    "mbtp"|"mousetapbackward"	=> return custom(CustomAction::MouseTap(Btn::Backward),&s.a),