mikelax / forgingadventures

0 stars 0 forks source link

closes #108 - Implement Custom Styles for Game Message Formats #146

Closed nazar closed 6 years ago

nazar commented 6 years ago

Closes #108

mikelax commented 6 years ago

hey @nazar this is looking good. Here are a couple of quick callouts that I noticed.

This next one I am on the fence about. If you have a Format selected (ie. Talking), and then hit the enter key. should that "end" the Format? ie. this is how the "Header" works. Don't make any change for this yet but give it some thought and we can discuss.

nazar commented 6 years ago

When editing a game message, the Formats dropdown is missing

fixed

Can we change what displays in the formats dropdown from "Formats" to "In Character Formats"

When a format is selected, can that given format be displayed in the dropdown. Its a little confusing now as for example the "B" icon is displayed as on, but you have to click again the Formats dropdown to see that "Talking" is selected

I looked through the tinyMCE docs for the styleselect toolbar control but couldn't find anything relevant for the above. Reading one SO answer suggests we might need to implement our own custom plugin to support this.

If you have a Format selected (ie. Talking), and then hit the enter key. should that "end" the Format?

I did add an option to apply the above in anything selected in the "paragraph" and "heading" list. This same reset, however, does not seem to be applying to anything selected from the Formats list.

We might need to dive deeper into the docs as the "Formats" menu might not be the best place to put our custom formatting? We just need to figure out how to add custom items to the formatselect toolbar list control.

nazar commented 6 years ago

I might be able to copy this formatselect plugin, which is used to show the "header" and "paragraph" options and extend it to add our own custom formats.

Maybe something to try in v1.x?

nazar commented 6 years ago

When a format is selected, can that given format be displayed in the dropdown. Its a little confusing now as for example the "B" icon is displayed as on

I might have a solution for this....

mikelax commented 6 years ago

@nazar cool, with the latest updates I think that we are in good shape. If there is an easy way to "not sync" the B, I, buttons when a Format is selected I think that will be good to call it for v1.

In a point release we can look to see about a custom plugin, etc. If we do switch up the Formats, that won't affect the styling for any already created posts, right?

nazar commented 6 years ago

If there is an easy way to "not sync" the B, I, buttons when a Format is selected I think that will be good to call it for v1

My last push fixed that 😉

nazar commented 6 years ago

In a point release we can look to see about a custom plugin, etc. If we do switch up the Formats, that won't affect the styling for any already created posts, right?

Yeah, I'm concerned about that too. I updated the format styling to only set class names so that 🤞 should minimise plugin transition issues...