jquery - 通过 jQuery 修改值后数据属性不应用 css

Question

我想按列表项 (li) 实现检查列表。我使用数据检查属性来保持检查状态。我使用 CSS 选择器设置图标颜色(“true”= 黑色，“false”= 白色)。

当页面显示时它工作正常，所有任务都显示白色图标，因为它们的默认数据检查值为“false”。但是当我点击列表项时，数据检查值被修改为'false'，但颜色没有变成黑色。

我已经通过提醒他们检查了值。他们是正确的。

我做错了什么？

谢谢。

HTML:

<ul>
  <li data-id='1' data-checked='false' onclick='toggle_task(this);'>
    <i class='fs fa-check'></i>
    <span>Task A</span>
  </li>
  <li data-id='2' data-checked='false' onclick='toggle_task(this);'>
    <i class='fs fa-check'></i>
    <span>Task B</span>
  </li>
</ul>
<button onclick='show_checked();'>Done</button>

CSS:

ul li[data-checked='true'] i:first-child {
  color: #000000;
}
ul li[data-checked='false'] i:first-child {
  color: #eeeeee;
}

Javascript:

function toggle_task(sender) {
  if ($(sender).data('checked').toString == 'true') {
    $(sender).data('checked','false');
  } else {
    $(sender).data('checked','true');
  }
}
function show_checked() {
  $(li).each(function() {
    var text = $(this).data('id').toString() + ': ' + $(this).data('checked').toString();
    alert($(this).data('checked'));
  });
}

Answer 1

我想你已经对 HTML data-* attributes 感到困惑了(您将其用作 CSS 选择器的一部分)和 jQuery data method它存储与元素关联但在 DOM 之外的任意 JavaScript 对象。困惑可能来自于此-

.data( key ) Returns: Object Description:

Return the value at the named data store for the first element in the jQuery collection, as set by data(name, value) or by an HTML5 data-* attribute.

该方法允许您从 HTML 5 data-* 属性读取数据(如果在 jQuery 集合中没有具有该键的值，但不能设置它)。

您要做的是使用 jQuery attr method像这样-

function toggle_task(sender) {
  if ($(sender).attr('data-checked') === 'true') {
    $(sender).attr('data-checked','false');
  } else {
    $(sender).attr('data-checked','true');
  }
}

function show_checked() {
  $('li').each(function() {
    var text = $(this).attr('id') + ': ' + $(this).attr('data-checked');
    alert($(this).attr('data-checked')); //Consider using console.log rather than alert here
  }); //Fixed your jQuery selector - required quotes for li element - is not a JavaScript variable
}

作为奖励，attr 方法返回一个字符串而不是一个对象 - 因此您可以免除 toString 调用。

我不完全确定你想在视觉上做什么，但是如果你想要一个彩色 block 需要元素指示它是否被选中(在这种情况下 元素不是最语义化的适当的)你还需要更新你的 CSS-

/* These need to be background-color, not color */

ul li[data-checked='true'] i:first-child {
  background-color: #000000;
}
ul li[data-checked='false'] i:first-child {
  background-color: #eeeeee;
}

/* Also need to add these styles to actually see the empty elements */

.fa-check {
    display: inline-block;
    height: 0.5em;
    width: 0.5em;
}

我创建了 your original code 的 JSFiddles和 the updated code这是我认为您想要的方式。

拜托，拜托，请考虑使用 jQuery on method 将不显眼的事件处理程序与 jQuery 结合使用而不是具有内联的“onclick”属性。这使您的 JavaScript 逻辑与 HTML 内容分开，请参阅 http://en.wikipedia.org/wiki/Unobtrusive_JavaScript .