Those shortcuts already work on the whole objects in the node tool (even
if the alignanddistributepanel does not show the buttons).


Huh, never noticed that. It's correct:
If you have two objects selected, and are editing both objects at once using the node tool, the align hotkeys align both shapes regardless of what nodes are selected.

This is an error imho. While in node editing mode it makes sense to leave the parent objects alone (since that's a select tool alignment operation).

There is a choice to be made between:
- either the current situation should be kept (those shortcuts/verbs
should work the same (aligning whole objects) at all times)
- either the shortcut aligning objects should align nodes in node tool

The same hotkeys for aligning objects in Select Tool mode should align nodes in Node tool mode. This is the most intuitive way to handle it.
If the user wants to align shapes with hotkeys they would have already done so in select mode. I'd keep node tool mode for editing nodes, and link alignment hotkeys accordingly.

-C

 
 
--
Mc

On 03/11/2016 07:04 PM, C R wrote:
> Ctrl-Alt-V it's already used for paste in place. Ctrl-alt-T is
> consistent with what is used to align objects. These alignment hotkeys
> should always be the same imho, regardless of what kind of things you
> are aligning. :)
>
> -C
>
> On 11 Mar 2016 5:10 pm, "Olof Bjarnason" <olof.bjarnason@...400...
> <mailto:olof.bjarnason@...400...>> wrote:
>
>     Yes this is my gut feeling too - not to hard code any keyboard
>     shortcuts because of the collision risk.
>
>     But I need a little help on how the custom shortcuts system functions...
>
>     E.g. in what source file can I read to learn?
>
>     On 11 Mar 2016 18:02, "LucaDC" <dicappello@...2144...
>     <mailto:dicappello@...2144...>> wrote:
>
>         Please, don't hardcode shortcuts with more than one modifier (like
>         Ctrl+Alt+<key>) because under Windows they could be used to
>         launch some
>         application so they should always be customizable.
>         Combinations with one or zero modifiers (like Ctrl+<key> or
>         Alt+<key>) are
>         usually application specific so it's not a problem if they are
>         hardcoded,
>         unless it's something weird like Ctrl-V to vertically align nodes :)
>
>         Luca
>
>
>
>
>         --
>         View this message in context:
>         http://inkscape.13.x6.nabble.com/Quick-fix-extend-object-alignment-hotkeys-to-work-for-nodes-tp4975985p4976007.html
>         Sent from the Inkscape - Dev mailing list archive at Nabble.com.
>
>         ------------------------------------------------------------------------------
>         Transform Data into Opportunity.
>         Accelerate data analysis in your applications with
>         Intel Data Analytics Acceleration Library.
>         Click to learn more.
>         http://pubads.g.doubleclick.net/gampad/clk?id=278785111&iu=/4140
>         _______________________________________________
>         Inkscape-devel mailing list
>         Inkscape-devel@lists.sourceforge.net
>         <mailto:Inkscape-devel@lists.sourceforge.net>
>         https://lists.sourceforge.net/lists/listinfo/inkscape-devel
>
>
>     ------------------------------------------------------------------------------
>     Transform Data into Opportunity.
>     Accelerate data analysis in your applications with
>     Intel Data Analytics Acceleration Library.
>     Click to learn more.
>     http://pubads.g.doubleclick.net/gampad/clk?id=278785111&iu=/4140
>     _______________________________________________
>     Inkscape-devel mailing list
>     Inkscape-devel@lists.sourceforge.net
>     <mailto:Inkscape-devel@lists.sourceforge.net>
>     https://lists.sourceforge.net/lists/listinfo/inkscape-devel
>
>
>
> ------------------------------------------------------------------------------
> Transform Data into Opportunity.
> Accelerate data analysis in your applications with
> Intel Data Analytics Acceleration Library.
> Click to learn more.
> http://pubads.g.doubleclick.net/gampad/clk?id=278785111&iu=/4140
>
>
>
> _______________________________________________
> Inkscape-devel mailing list
> Inkscape-devel@lists.sourceforge.net
> https://lists.sourceforge.net/lists/listinfo/inkscape-devel
>


------------------------------------------------------------------------------
Transform Data into Opportunity.
Accelerate data analysis in your applications with
Intel Data Analytics Acceleration Library.
Click to learn more.
http://pubads.g.doubleclick.net/gampad/clk?id=278785111&iu=/4140
_______________________________________________
Inkscape-devel mailing list
Inkscape-devel@...1794...s.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/inkscape-devel