Commit 567f4aabe10e6ad1dd832c17fb584de1f5b3fc14
Committed by
Álvaro Fernando Matos de Souza
1 parent
86320e27
Exists in
master
and in
5 other branches
Add tooltop span and fix javascript show/hide method
Signed-off-by: Fábio Teixeira <fabio1079@gmail.com>
Showing
3 changed files
with
18 additions
and
11 deletions
Show diff stats
public/style.css
public/views/comments-software-extra-fields.js
| ... | ... | @@ -5,26 +5,26 @@ modulejs.define('CommentsSoftwareExtraFields', ['jquery'], function($) { |
| 5 | 5 | information_display_state: "hidden" |
| 6 | 6 | } |
| 7 | 7 | |
| 8 | - function set_show_aditional_information() { | |
| 9 | - $(".comments-software-extra-fields span").on("click", function() { | |
| 8 | + function set_show_additional_information() { | |
| 9 | + $("#comments-additional-information").on("click", function() { | |
| 10 | 10 | if (DATA.information_display_state === "hidden") { |
| 11 | 11 | DATA.information_display_state = "show"; |
| 12 | - $(this).parent().children("div").show(); | |
| 12 | + $(".comments-software-extra-fields div").show(); | |
| 13 | 13 | } else { |
| 14 | 14 | DATA.information_display_state = "hidden"; |
| 15 | - $(this).parent().children("div").hide(); | |
| 15 | + $(".comments-software-extra-fields div").hide(); | |
| 16 | 16 | } |
| 17 | 17 | }); |
| 18 | 18 | } |
| 19 | 19 | |
| 20 | 20 | return { |
| 21 | 21 | isCurrentPage: function() { |
| 22 | - return $(".comments-software-extra-fields span").length === 1; | |
| 22 | + return $(".star-rate-form").length === 1; | |
| 23 | 23 | }, |
| 24 | 24 | |
| 25 | 25 | |
| 26 | 26 | init: function() { |
| 27 | - set_show_aditional_information(); | |
| 27 | + set_show_additional_information(); | |
| 28 | 28 | } |
| 29 | 29 | } |
| 30 | 30 | }); | ... | ... |
views/comments_extra_fields.html.erb
| 1 | -<div class="comments-software-extra-fields"> | |
| 2 | - <span> | |
| 3 | - <%= _("Aditional informations") %> | |
| 1 | +<div class="comments-display-fields"> | |
| 2 | + <span id="comments-additional-information"> | |
| 3 | + <%= _("Additional informations") %> | |
| 4 | 4 | </span> |
| 5 | 5 | |
| 6 | + <span class="comments-arrow-down"></span> | |
| 7 | +</div> | |
| 8 | + | |
| 9 | +<div class="comments-software-extra-fields"> | |
| 6 | 10 | <div class="comments-software-people-benefited"> |
| 7 | 11 | <%= label_tag "comments_people_benefited", _("Number of Beneficiaries")%> |
| 12 | + <span class="star-tooltip"></span> | |
| 8 | 13 | <%= text_field_tag "comments[people_benefited]", "" %> |
| 9 | 14 | </div> |
| 15 | + | |
| 10 | 16 | <div class="comments-software-saved-values"> |
| 11 | 17 | <%= label_tag "comments_saved_value", _("Saved resources")%> |
| 12 | - <%= text_field_tag "comments[saved_value]", ""%> | |
| 18 | + <span class="star-tooltip"></span> | |
| 19 | + <%= text_field_tag "comments[saved_value]", "", :placeholder=>"R$"%> | |
| 13 | 20 | </div> |
| 14 | 21 | </div> | ... | ... |