vassili/seitime-frappe
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions 1

refactor: move the css classes for star rating from website saas to control less file

Browse source
This commit is contained in:
Mangesh-Khairnar 2019-03-25 15:10:43 +05:30
parent 4e0dab3621
commit c9f8c4e40e
3 changed files with 26 additions and 23 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

28
frappe/public/js/frappe/form/controls/rating.js
View file

@ -3,11 +3,11 @@ frappe.ui.form.ControlRating = frappe.ui.form.ControlInt.extend({
this._super();
const star_template = `
<div class = "rating">
<i class="fa fa-fw fa-star text-extra-muted" data-idx=1></i>
<i class="fa fa-fw fa-star text-extra-muted" data-idx=2></i>
<i class="fa fa-fw fa-star text-extra-muted" data-idx=3></i>
<i class="fa fa-fw fa-star text-extra-muted" data-idx=4></i>
<i class="fa fa-fw fa-star text-extra-muted" data-idx=5></i>
<i class="fa fa-fw fa-star" data-idx=1></i>
<i class="fa fa-fw fa-star" data-idx=2></i>
<i class="fa fa-fw fa-star" data-idx=3></i>
<i class="fa fa-fw fa-star" data-idx=4></i>
<i class="fa fa-fw fa-star" data-idx=5></i>
</div>
`;
@ -15,29 +15,29 @@ frappe.ui.form.ControlRating = frappe.ui.form.ControlInt.extend({
this.$input_wrapper.find('i').hover((ev) => {
const el = $(ev.currentTarget);
var star_value = el.data('idx');
let star_value = el.data('idx');
el.parent().children('i.fa').each( function(e){
if (e < star_value) {
$(this).addClass('hover');
$(this).addClass('star-hover');
} else {
$(this).removeClass('hover');
$(this).removeClass('star-hover');
}
});
}, (ev) => {
const el = $(ev.currentTarget);
el.parent().children('i.fa').each( function(e) {
$(this).removeClass('hover');
$(this).removeClass('star-hover');
});
});
this.$input_wrapper.find('i').click((ev) => {
const el = $(ev.currentTarget);
var star_value = el.data('idx');
let star_value = el.data('idx');
el.parent().children('i.fa').each( function(e) {
if (e < star_value){
$(this).addClass('click');
$(this).addClass('star-click');
} else {
$(this).removeClass('click');
$(this).removeClass('star-click');
}
});
this.set_value(star_value);
@ -47,10 +47,10 @@ frappe.ui.form.ControlRating = frappe.ui.form.ControlInt.extend({
return this.value ? this.value : 0;
},
set_formatted_input(value) {
var el = this.$input_wrapper.find('i');
let el = this.$input_wrapper.find('i');
el.children('i.fa').prevObject.each( function(e) {
if (e < value){
$(this).addClass('click');
$(this).addClass('star-click');
}
});
}

12
frappe/public/less/controls.less
View file

@ -77,3 +77,15 @@
margin-right: 5px;
margin-bottom: 5px;
}
.rating {
i {
color: @text-extra-muted;
}
.star-hover {
color: @text-muted;
}
.star-click {
color: @text-light;
}
}

9
frappe/public/scss/website.scss
View file

@ -90,13 +90,4 @@ img {
.indicator {
font-size: inherit;
}
.rating {
.hover {
color: rgb(148, 148, 143) !important;
}
.click {
color: rgb(7, 7, 6) !important;
}
}