CVS edit link for rv0

I made a module which integrates with the Wysiwyg module to provide the functionality of re-ordering buttons and adding separators between them.
This is a much requested feature (see: http://drupal.org/node/277954 ) and from reading the issue I see that plans are underway to integrate such functionality into future versions (under D7). However there is still a long way to go for that, hence the creation of this module for D6. From what is said in the above issue it is quite clear that a module like mine should be kept separate from the main Wysiwyg code for now.
As for dependancies: only Wysiwyg module is needed. Currently TinyMCE is supported, but I'm planning to add FCKEditor if there is enough demand for it.
CommentFileSizeAuthor
#1 wysiwyg_buttonorder.zip6.06 KBrv0

Comments

rv0’s picture

rv0 commented
Title: Wysiwyg Button Order [rv0] » rv0 [rv0]
Project: Drupal.org security advisory coverage applications » Drupal.org CVS applications
Component: new project application » miscellaneous
Status: Needs review » Postponed (maintainer needs more info)
StatusFileSize
new wysiwyg_buttonorder.zip6.06 KB

Added: the module

EDIT: There is a newer version in the GIT repository, please get the latest version from GIT: http://drupal.org/node/1076396/revisions

rv0’s picture

rv0 commented
Component: Miscellaneous » new project application
Status: Postponed (maintainer needs more info) » Needs review
arianek’s picture

arianek commented
Status: Needs review » Postponed

Hi. Please read all the following and the links provided as this is very important information about your CVS Application:

Drupal.org has moved from CVS to Git! This is a very significant change for the Drupal community and for your application. Please read the following documentation on how this affects and benefits you and the application process:
Migrating from CVS Applications to (Git) Full Project Applications

  • The status of this application will be put to "postponed" and by following the instructions in the above link, you will be able to reopen it.
  • Or if your application has been "needs work" for more than 5 weeks, your application will be marked as "closed (won't fix)". You can still reopen it, by reading the instructions above.
rv0’s picture

rv0 commented
Title: rv0 [rv0] » Wysiwyg Button Order [rv0]
Project: Drupal.org CVS applications » Drupal.org security advisory coverage applications
Status: Postponed » Needs review

Uploaded to GIT repository:
http://drupal.org/sandbox/rv0/1076396

updates: changed project name: from wysiwyg_buttonorder to wysiwyg_button_order

looking forward to the module review, thanks

jippie1948’s picture

jippie1948 commented

Works great! Thank you very much, as this module was needed very strongly.

JPH

Anonymous’s picture

Anonymous (not verified) commented

Very useful. Works fine. If I have a suggestion it would be another separator which pushes all subsequent buttons onto the next line - but that would be icing on the cake.
Many thanks,
Tony.

rv0’s picture

rv0 commented

@jippie1948, @maynardsmith
Thanks for testing this module!

@maynardsmith
Useful suggestion and it's definitely possible with TinyMCE (as it has the unused functionality to make multiple rows)
As a workaround for now, I think you can use css on the first or last button in a row (using clear?)

I'm pretty sure the current module is stable as I've tested it a lot. But if you notice any bugs or anything, please post back here.

It would be nice to get this module out of the sandbox ;)

Frederic wbase’s picture

Frederic wbase commented
Title: rv0 [rv0] » Wysiwyg Button Order [rv0]
Project: Drupal.org CVS applications » Drupal.org security advisory coverage applications
Component: miscellaneous » new project application
Status: Postponed (maintainer needs more info) » Needs review

Verry good job. Works fine, it fulfils the expectations!

attiks’s picture

attiks commented
Status: Needs review » Reviewed & tested by the community

looks good to me

mlncn’s picture

mlncn commented
Status: Reviewed & tested by the community » Fixed

Sweet! Look forward to trying this one out.

rv0, you now have "vetted" status. Vetted users create full projects and promote existing experimental sandboxes.

Nice work and when you would like further review for any module please do ask for one, per http://groups.drupal.org/peer-review/requests

rv0’s picture

rv0 commented

thanks all!

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.