Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
Both version 2.3 and 3.x dev is included.
Both version 2.3 and 3.x dev is included.
Comments
Comment #1
marvil07 CreditAttribution: marvil07 commentedquoting #1002300-3: Contribution: additional voting widget - "thumbs up and down":
IMHO this is mainly the same as the upanddown widget, but using thumbs, that is why my original position about having a widget that do the same but with different style sheets do not worth to have it there.
But, I see that some people is interested on having a thumbs widget inside the module, and it's great that you have provided widgets for 6.x-2.x and 6.x-3.x versions :-), so instead of marking this as duplicate of the mentioned issue, I am considering adding it to 6.x-3.x.
So, what is needed to have it inside:
Comment #2
udvranto CreditAttribution: udvranto commentedGimme some time. I am using these GPLed SVG icons to create the new ones.
http://commons.wikimedia.org/wiki/File:Thumb_up_icon.svg
http://en.wikipedia.org/wiki/File:Thumb_down_icon.svg
Comment #3
udvranto CreditAttribution: udvranto commentedI created a svg using the gpl version of the thumbs. But the border is still too thick. Will this do?
http://en.wikipedia.org/wiki/File:Symbol_thumbs_down.svg
http://en.wikipedia.org/wiki/File:Symbol_thumbs_up.svg
Comment #4
Juan C CreditAttribution: Juan C commentedSubscribe
Comment #5
davethedruper CreditAttribution: davethedruper commentedSubscribe +2
Comment #6
franzkewd CreditAttribution: franzkewd commented+1
Comment #7
mcurry CreditAttribution: mcurry commentedsubscribe. I'd say +1 but last time I did that a maintainer nearly bit my finger off :D
Comment #8
mcurry CreditAttribution: mcurry commentedNewbie question: I'm not sure what I need to do with the files in the attached .zip archive. Do I dump the .tpl.php file into my theme directory? Or do I put all the files into the vote_up_down module dir? Or add it to the vote_up_down/widgets dir?
Comment #9
mcurry CreditAttribution: mcurry commentedOk, after some poking around, it looks like I need to create a directory under vote_up_down/widgets, copy the appropriate files into that new dir, then follow the instructions in vote_up_down/WIDGETAPI.txt to enable it.
Comment #10
marvil07 CreditAttribution: marvil07 commentedchanging status, hopefully I could make the final touches to get the widget done
Comment #11
fyrebear CreditAttribution: fyrebear commentedsubscribe
Comment #12
marvil07 CreditAttribution: marvil07 commentedNo more features for 6.x-3.x now that it is the stable branch, moving to 7.x-1.x as postponed until basic port is ready.
Comment #13
ezra-g CreditAttribution: ezra-g commentedI discussed with marvil07 who clarified that with an SVG, he would accept a patch to provide a thumbs up/down widget. Leaving as postponed pending a patch ;).
Comment #14
ay13 CreditAttribution: ay13 commentedThumbs up down widget patch attached.
Comment #15
ezra-g CreditAttribution: ezra-g commentedNote, this implements the widget in the design at http://drupal.org/files/q-a-v2.gif from #1392768: question and answers (Q&A) feature & Origins theme tweaks.
Comment #16
marvil07 CreditAttribution: marvil07 commentedThe widget looks awesome!
There is some work to do on it yet, but that's code, not design ;-)
So, let's see what I can do now.
Comment #17
marvil07 CreditAttribution: marvil07 commentedSo, quite a lot of changes:
What's missing here(or why this is still in "needs work" status):
Feel free to do that change. I will be back on this issue hopefully soon.
Comment #18
ezra-g CreditAttribution: ezra-g commentedThanks for making some enhancements!
We're planning to separately build a "thumbs up" only widget. So, naming this widget "thumbs" wouldn't distinguish to two. Perhaps we should undo this change? Or, could you explain the motivation for this change so that we can think of a solution?
Our motivation here is to have the widget function as per the comp at http://drupal.org/files/q-a-v2.gif, which shows the net result of the points in a single, easy to understand number. I recognize that it's different from how the existing widgets display, but this seems like a benefit of additional widgets :). The change you're proposing is here is fundamental to how the widget functions. Speaking for myself and ay13, our motivation is to meet the requirements in the comp in a way that contributes back to VUD, and changing that aspect of the widget is kind of a dealbreaker for our project. Perhaps we can discuss in IRC :)?
Comment #19
marvil07 CreditAttribution: marvil07 commentedPlease notice that what I changed is only the internal code name. And the reasoning behind it is just to make it shorter the names. Hopefully that's OK for you.
About the second point, well, it will be always possible to create another widget, so let's do it in that way.
@ezra-g: I will be then pushing the last patch, but before it will be great to hear from you.
Comment #20
ezra-g CreditAttribution: ezra-g commentedUnderstood - Let me rephrase my question: We're about to start working on a patch for another thumbs widget that only allows voting up. If we assign the machine name 'thumbs' to the present widget, what would be an appropriate name for the new one?
Comment #21
ezra-g CreditAttribution: ezra-g commentedAlso, it seems like you changed these from being red and green to being grey and black. Was this intentional?
Comment #22
scthomps312 CreditAttribution: scthomps312 commentedSubscribe
Comment #23
bdone CreditAttribution: bdone commented"subscribe" comments on Drupal.org issues are no longer needed.
please see: http://drupal.org/node/1306444
Comment #24
marvil07 CreditAttribution: marvil07 commentedNot sure about including another widget for thumbs up in vud. The idea about widgets included here is not about filling all possible widgets, but instead about showing what it could be done with the ctools widget api.
I guess that this will be easier if I move ctools plugin implementation to OOP plugins, so a thumbs_up widget can inherit most functionality from thumbs widget, but sadly that's not possible right now, and I would say that should go only on 7.x
Yes, as mentioned on comment 17:
This is also because IMHO using grayscale will help a little more to avoid implementing a custom widget for changing colours(to match their themes). IMHO grayscale is more neutral.
Comment #25
ezra-g CreditAttribution: ezra-g commentedWe chose red and green because these are standard UI colors to indicate positive and negative and therefore seems unlikely to need re-implementation on a per-theme basis. http://drupal.org/node/871284 shows that VUD already ships with a red and green widget, so I'm not clear on why the presently proposed widget needs this change.
As stated earlier the thread, this widget is a dependency for a new feature in the Commons distribution #1392768: question and answers (Q&A) feature & Origins theme tweaks. Our options for implementing the widget were to do so within the Commons distribution or as a patch to VUD - We chose a patch because that would have broader benefit for folks who might not be running Commons. Making significant UI change to the widget means it no longer meets our goals for Commons.
Given that and
, perhaps the best way to make these widgets broadly available would be for us to create a contributed VUD_Extra module - Let me know if that seems like a preferable solution and we'll create a new contrib :).
Comment #26
marvil07 CreditAttribution: marvil07 commentedOh, I did not change that image yet, ggrrhh
That image was taken from 6.x-2.x widgets.
The long explanation is on #816776: better colours for plain, alternate and upanddown widgets, and it's about accessibility.
Hopefully you also agree on the colour now you know the real reason which I just failed to mention before.
Now that we are so close I would really like to add this widget on vud, but another thumbs_up widget IMHO is more than needed. Actually IIRC I have said no to other widgets because of the same reason, so if someone is willing to maintain/add those widgets and change all them accordingly to the api changes I made it would be great to have one place to point people wanting to add more widgets.
Comment #27
ezra-g CreditAttribution: ezra-g commentedFollowing discussion with marvil07 in IRC, I'm moving this to the Commons queue to create a standalone contrib module that provides the widgets for #1392768: question and answers (Q&A) feature & Origins theme tweaks and #1392756: Create ideation feature (designs attached).
Comment #28
marvil07 CreditAttribution: marvil07 commentedI will be pushing the patch on comment 17 into 6.x-3.x and 7.x-1.x so the vud colours thumbs widget there can also live somewhere.
I have also created #1426398: Convert vud widgets into D8 plugins for 7.x as the long term solution for inheritance between widgets.
Comment #29
marvil07 CreditAttribution: marvil07 commentedJust to mention that after porting to D7 I have just pushed the patches attached here(I've added a minor change, see interdiff) on 6.x-3.x and 7.x-1.x.
Comment #30
lightsurge CreditAttribution: lightsurge commented@marvil07
Your local path seems to have snuck through here:
Comment #31
ezra-g CreditAttribution: ezra-g commentedIt looks like going to prioritize upcoming work on updating Commons to Drupal 7, and as a result we're going to hold off on implementing the Ideation feature.
As a result, it seems reasonable to simply alter the image used in the thumbs widget via css to get our red/green colors.
The attached patch implements this.
Thanks to ay13 for providing some CSS tips about the background-image property.
Comment #32
ezra-g CreditAttribution: ezra-g commentedThis is committed. http://drupalcode.org/project/commons.git/commitdiff/ec2d7cd?hp=16001111...