github-mirror/discourse
2
0
Fork 0
mirror of https://github.com/discourse/discourse.git synced 2024-11-22 09:42:02 +08:00
Code Issues Actions 1 Packages Projects Releases Wiki Activity

FIX: Workaround limitation in jquery.autoellipsis (#8747)

Browse Source 
Calling $.ellipsis() on an element containing <br> elements would throw
an exception.
This commit is contained in:
Bianca Nenciu 2020-01-21 18:44:50 +02:00 committed by GitHub
parent 0a27086764
commit 9c628f0897
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
2 changed files with 33 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

1
app/assets/javascripts/discourse/components/text-overflow.js.es6
View File

@ -7,6 +7,7 @@ export default Component.extend({
const $this = $(this.element);
if ($this) {
$this.find("br").replaceWith(" ");
$this.find("hr").remove();
$this.ellipsis();
}

32
test/javascripts/components/text-overflow-test.js.es6 Normal file
View File

@ -0,0 +1,32 @@
import componentTest from "helpers/component-test";
moduleForComponent("text-overflow", { integration: true });
componentTest("default", {
template: `
<style>
.overflow {
max-height: 40px;
overflow: hidden;
width: 500px;
}
</style>
<div>{{text-overflow class='overflow' text=text}}</div>`,
beforeEach() {
this.set(
"text",
"Lorem ipsum dolor sit amet, consectetur adipiscing elit.\nFusce convallis faucibus tortor quis vestibulum.<br>\nPhasellus pharetra dolor eget imperdiet tempor.<br>\nQuisque hendrerit magna id consectetur rutrum.<br>\nNulla vel tortor leo.<br>\nFusce ullamcorper lacus quis sodales ornare.<br>"
);
},
test(assert) {
assert.equal(
find(".overflow")
.text()
.trim(),
"Lorem ipsum dolor sit amet, consectetur adipiscing elit.\nFusce convallis faucibus tortor quis vestibulum. Phasellus pharetra dolor eget imperdiet..."
);
}
});