spohlenz / tinymce-rails

Integration of TinyMCE with the Rails asset pipeline
Other
815 stars 256 forks source link

Fix link for config options in README #196

Closed jmeridth closed 8 years ago

jmeridth commented 8 years ago
spohlenz commented 8 years ago

Thanks for letting me know. While not an exact match to the previous page, it seem like https://www.tinymce.com/docs/configure/ might be a better fit for the link target, since each sub-page goes into detail about each of the available options (and not just the toolbar options). What do you think?

jmeridth commented 8 years ago

@spohlenz I'm still leaning towards my suggestion only because as a first-time user, in the context of where the link is in the README, they just want to know what other config options can go into config/tinymce.yml file. Exactly like I did :smile: This allows them (and myself) to get up and running quicker. Maybe a second link to the docs in general afterwards or before?

From https://www.tinymce.com/docs/configure/ the user would have to click Explore Advanced Topics in the left menu and then Editor Control Identifiers. That is not intuitive for first-time users.

wdyt?

spohlenz commented 8 years ago

The documentation is a bit more confusing now, but it looks like the Editor Control Identifiers page is just the toolbar/menu options, whereas the subpages of the Configure section go into detail on all of the configuration options that are available, each of which can be configured in tinymce.yml.

It's unfortunate that there doesn't seem to a list of all of the options on one page anymore, but I'm still inclined towards https://www.tinymce.com/docs/configure/ being the best fit.

jmeridth commented 8 years ago

@spohlenz link changed to match your request. I agree, with all of those options being available, the general link is better. Cheers. :beers:

spohlenz commented 8 years ago

Merged. Thank you.