ProjectSegfault/invidious-experimenting
4
0
Fork 0
forked from midou/invidious
Code Pull Requests Activity

Limit descriptions width to ease mixed LTR/RTL text reading

Browse Source 
This will prevent, on large pages, the LTR and RTL text to be
far away, on each side of the page. This could happen on channel
and playlists descriptions, when the page is displayed on a large
screen.
This commit is contained in:
Samantaz Fox 2021-06-24 15:00:55 +02:00 committed by Mostafa Ahangarha
parent d16a748f37
commit 9e4fd193c6
6 changed files with 19 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
assets/css/default.css
View File

@ -523,3 +523,7 @@ p,
unicode-bidi: plaintext; unicode-bidi: plaintext;
text-align: start; text-align: start;
} }
#descriptionWrapper {
max-width: 600px;
}

2
src/invidious/views/channel.ecr
View File

@ -28,8 +28,10 @@
</div> </div>
<div class="h-box"> <div class="h-box">
<div id="descriptionWrapper">
<p><span style="white-space:pre-wrap"><%= channel.description_html %></span></p> <p><span style="white-space:pre-wrap"><%= channel.description_html %></span></p>
</div> </div>
</div>
<div class="h-box"> <div class="h-box">
<% ucid = channel.ucid %> <% ucid = channel.ucid %>

2
src/invidious/views/community.ecr
View File

@ -27,8 +27,10 @@
</div> </div>
<div class="h-box"> <div class="h-box">
<div id="descriptionWrapper">
<p><span style="white-space:pre-wrap"><%= XML.parse_html(channel.description_html).xpath_node(%q(.//pre)).try &.content %></span></p> <p><span style="white-space:pre-wrap"><%= XML.parse_html(channel.description_html).xpath_node(%q(.//pre)).try &.content %></span></p>
</div> </div>
</div>
<div class="h-box"> <div class="h-box">
<% ucid = channel.ucid %> <% ucid = channel.ucid %>

2
src/invidious/views/playlist.ecr
View File

@ -64,8 +64,10 @@
</div> </div>
<div class="h-box"> <div class="h-box">
<div id="descriptionWrapper">
<p><%= playlist.description_html %></p> <p><%= playlist.description_html %></p>
</div> </div>
</div>
<% if playlist.is_a?(InvidiousPlaylist) && playlist.author == user.try &.email %> <% if playlist.is_a?(InvidiousPlaylist) && playlist.author == user.try &.email %>
<div class="h-box" style="text-align:right"> <div class="h-box" style="text-align:right">

2
src/invidious/views/playlists.ecr
View File

@ -27,8 +27,10 @@
</div> </div>
<div class="h-box"> <div class="h-box">
<div id="descriptionWrapper">
<p><span style="white-space:pre-wrap"><%= XML.parse_html(channel.description_html).xpath_node(%q(.//pre)).try &.content if !channel.description_html.empty? %></span></p> <p><span style="white-space:pre-wrap"><%= XML.parse_html(channel.description_html).xpath_node(%q(.//pre)).try &.content if !channel.description_html.empty? %></span></p>
</div> </div>
</div>
<div class="h-box"> <div class="h-box">
<% ucid = channel.ucid %> <% ucid = channel.ucid %>

2
src/invidious/views/watch.ecr
View File

@ -246,7 +246,9 @@ we're going to need to do it here in order to allow for translations.
<div id="description-box"> <!-- Description --> <div id="description-box"> <!-- Description -->
<% if video.description.size < 200 || params.extend_desc %> <% if video.description.size < 200 || params.extend_desc %>
<div id="descriptionWrapper">
<%= video.description_html %> <%= video.description_html %>
</div>
<% else %> <% else %>
<input id="descexpansionbutton" type="checkbox"/> <input id="descexpansionbutton" type="checkbox"/>
<div id="descriptionWrapper"> <div id="descriptionWrapper">