View Poll Results: Do you think right click funtionality is a good idea?

Voters
197. You may not vote on this poll
  • Yes

    181 91.88%
  • No

    6 3.05%
  • Maybe

    10 5.08%
  • Don't Care / Know

    0 0%
+ Reply to Thread
Page 2 of 16 FirstFirst 1 2 3 4 12 ... LastLast
Results 11 to 20 of 154

Thread: Right Click Functionality

  1. #11
    lacri's Avatar
    lacri is offline Registered User
    Join Date
    Jan 2009
    Location
    Lutherstadt Wittenberg, Germany
    Posts
    179
    Downloads
    5
    Uploads
    0

    Default

    I have found a solution.

    add in contextmenu.js after the line
    PHP Code:
    rcmail.add_onload('rcm_contextmenu_load()'); 
    this
    PHP Code:
    rcmail.addEventListener('listupdate', function(evtprops) { rcm_contextmenu_load(); } ); 
    with the Event listupdate reloads the contextmenu_load function and becomes the correct env.mailbox var to display the correct delete Message in context.

  2. #12
    JohnDoh is offline Super Moderator
    Join Date
    May 2007
    Posts
    1,206
    Downloads
    6
    Uploads
    0

    Default

    thanks lacri, i think the delete function really doesnt like me very much. I have made a fix similar to your suggestion. the new version is available now. hopefully that is all the bugs squished.....
    Roundcube Patches: Sieverules, SpamAssassin Prefs, and more…

  3. #13
    lacri's Avatar
    lacri is offline Registered User
    Join Date
    Jan 2009
    Location
    Lutherstadt Wittenberg, Germany
    Posts
    179
    Downloads
    5
    Uploads
    0

    Default

    Thanks John works realy good, only the Internet Explorer not like this and will not display the context menu after klicking on a folder.

    i use the listupdate Event to reinitilize the rcm_contextmenu_init('messagelist tbody tr');
    PHP Code:
    rcmail.addEventListener('listupdate', function(evtprops) { rcm_contextmenu_init('messagelist tbody tr'); } ); 
    and after this works fine in all browsers for me i hope this is correct
    Last edited by lacri; 04-30-2009 at 01:05 PM.

  4. #14
    JohnDoh is offline Super Moderator
    Join Date
    May 2007
    Posts
    1,206
    Downloads
    6
    Uploads
    0

    Default

    thanks again lacri. I have made the change as you suggested. I am not overly happy with it as i think it means the events are added twice in most cases but i dont have time to look for a better solution right now, if any one has any suggestions please let me know.
    Roundcube Patches: Sieverules, SpamAssassin Prefs, and more…

  5. #15
    mfallon is offline Registered User
    Join Date
    Jul 2008
    Posts
    15
    Downloads
    5
    Uploads
    0

    Default Great Work!

    Really nice to see suggestions being implemented, things like this are what makes Roundcube (IMHO) the best opensource webmail client available.

    Keep up the good work guys.

  6. #16
    uberlemurguy is offline Registered User
    Join Date
    Aug 2008
    Posts
    4
    Downloads
    0
    Uploads
    0

    Default

    i honestly believe that using jQuery would simplify much of this complex javascript code in roundcube. There would be significantly less code, that is also easier to read. We could need things like: "return rcmail.folder_mouse_up('[Gmail]/All Mail')" on every folder link because we could bind classes with jQuery. jQuery has excellent AJAX methods and it works cross browser. There are hundreds of plugins easily enabling right-click functionality. Also skins could take advantage of jQuery, creating more dynamic and rich themes. I know all of my themes that I am going to make will include jQuery because of its advantages. If this has been discussed before, I am sorry because I am new to the forums (but not to RoundCube).

  7. #17
    JohnDoh is offline Super Moderator
    Join Date
    May 2007
    Posts
    1,206
    Downloads
    6
    Uploads
    0

    Default

    jQuery is being used in the trunk and will be in the next release i think, but there are some things that (at the moment) jQuery is slow at so it cant be used for everything. there is a little bit of discussion here RoundCube Mailing Lists

    as for the right click stuff you are correct, that is why i waited until RC was jQueryfied before i did any thing.

    i have been tinkering with RC for a long time now and IMO the JS is great and its really easy to understand once you get your head round the basic structure.
    Roundcube Patches: Sieverules, SpamAssassin Prefs, and more…

  8. #18
    uberlemurguy is offline Registered User
    Join Date
    Aug 2008
    Posts
    4
    Downloads
    0
    Uploads
    0

    Default

    I have used jQuery for over a year, and I have never run into a case where jQuery's ajax methods are slower. There must have been some other varible involved as in not using proper jQuery techniques etc.

    Also with jQuery even if it is a little slower, I don't think that should put us off of it. The ability to easily bind keys and run animations with ease is rather helpful. It also allows us to do more mundane things that are useful. (adding odd and even classes to the message list can be just 2 or 3 lines) It does look like the devs have done a good job with the rcmail object, but so much code could be elimitated by effectively using jQuery.

    Just my two cents.

  9. #19
    uberlemurguy is offline Registered User
    Join Date
    Aug 2008
    Posts
    4
    Downloads
    0
    Uploads
    0

    Default

    as an update: i did some tests myself, jQuery is just as fast at ajax after I ran my own tests, which makes sense since the jQuery $.ajax method is just a way to easily do the DOM ajax.

  10. #20
    rosali's Avatar
    rosali is offline Super Moderator
    Join Date
    Dec 2007
    Location
    Germany
    Posts
    2,394
    Downloads
    36
    Uploads
    0

    Default

    Great plugin! Only thing I miss, is the message row is selected (displayed in red colour) on right click.
    Regards,
    Rosali

    __________________
    MyRoundcube Project http://myroundcube.googlecode.com
    MyRoundcube Online Demo - Free Email Address http://mail4us.net
    MyRoundcube Plugins Generic Installation Guide http://mail4us.net/myroundcube/index.php
    Mailing List http://mail4us.net/?_action=plugin.nabble

+ Reply to Thread
Page 2 of 16 FirstFirst 1 2 3 4 12 ... LastLast

LinkBacks (?)

  1. 06-11-2009, 03:27 PM

Thread Information

Users Browsing this Thread

There are currently 1 users browsing this thread. (0 members and 1 guests)

     

Posting Permissions

  • You may not post new threads
  • You may not post replies
  • You may not post attachments
  • You may not edit your posts