On 14 Nov 2018, at 19:07, Tim Bates wrote:
It would be nice if the Align Assignments command absorbed the smart code from the align bundle, which appears moribund (?), but which is much smarter about alignments.
On 14 Nov 2018, at 12:34 pm, Allan Odgaard mailinglist@textmate.org wrote: I’m not keen on switching to a preferneces based system because majority of users will not want to customize the Align Assignments command, and I am also not sure what cases (handled by the default command) where the user would actually want to change it. But it would be useful to know where the current command gets it wrong or how it can be improved.
For one, given this selection in an R script:
#' selDVs = c("ht1", "ht2") #' mzData <- twinData[twinData$zygosity %in% "MZFF", ] #' dzData <- twinData[twinData$zygosity %in% "DZFF", ]
align does the right thing, and aligns the <- assignments. The built-in align does nothing. So having the built-in do the right thing here would be good.
The preferences system could be independent of user intervention, no? Just get bundle maintainers to include an align preference. Like this one for R And the ready-made align command in TM could inherit/import these.
{ shellVariables = ( { name = 'TM_SOURCE_ALIGNMENT_PATTERN'; value = '[ { "comment" : "align on assignments", "regexp" : "=|<-|->", "minimum_spacing_before" : " ", "minimum_spacing_after" : " ", "padding" : "before" }, { "comment" : "align right-hand side formulae assignments", "regexp" : "~", "minimum_spacing_before" : " ", "minimum_spacing_after" : " ", "padding" : "before" }, { "comment" : "align any stacked assignments on a line", "regexp" : ";", "minimum_spacing_before" : "", "minimum_spacing_after" : " ", "padding" : "before" }, { "comment" : "align any comments", "regexp" : "#", "minimum_spacing_before" : " ", "minimum_spacing_after" : " ", "padding" : "before" }]'; }, ); }