Ticket #1538 (closed enhancement: fixed)

Opened 3 years ago

Last modified 3 years ago

TableOperations: add config option to control currently-forced linebreak in toolbar

Reported by: ejucovy Owned by: ejucovy
Priority: normal Milestone: 0.97
Component: Plugins Version: trunk
Severity: normal Keywords: toolbar
Cc:

Description

The TableOperations plugin always adds a linebreak to the toolbar before appending its table buttons; its code contains:

var toolbar = ["linebreak", "inserttable", "toggleborders"];

In my application I prefer a layout where the table buttons are on the same line as the buttons that precede them.

The following patch extends the existing Xinha.Config.TableOperations configuration object to let the user control whether this forced-linebreak behavior is active.

Attachments

tablelinebreak.diff (1.6 kB) - added by guest 3 years ago.

Change History

Changed 3 years ago by guest

Changed 3 years ago by ejucovy

  • cc ejucovy@… removed
  • owner set to ejucovy

To clarify: my configuration also uses cfg.flowToolbars = false.

Changed 3 years ago by ejucovy

  • status changed from new to closed
  • resolution set to fixed

Since it doesn't *force* a linebreak unless flowToolbars is set to false, forceToolbarLineBreak is badly named. I renamed it to addToolbarLineBreak and committed in r1282.

Changed 3 years ago by ejucovy

  • keywords toolbar added
  • reporter changed from guest to ejucovy

Hmm, I do just want to add that this is a pretty silly solution -- but I can't think of a better one. There are a couple of issues that make me feel like an overhaul of the toolbar system would be useful -- some kind of alternative configuration that would let the user (meaning, the person who configures Xinha) *fully* specify the toolbar, after all plugins have registered their buttons. Among other things, this would require some kind of sensible API for querying the available buttons. It would also have to be fully backwards-compatible and let plugins continue to try to register "sensible default" locations in the way they do now. And, I have no idea what it would look like for the end-user.

So this is obviously a bigger project. I'm going to start tagging tickets with the keyword "toolbar" to keep track of the various use-cases as I see them.

Changed 3 years ago by ejucovy

  • milestone set to 0.97
Note: See TracTickets for help on using tickets.