-
Notifications
You must be signed in to change notification settings - Fork 2.1k
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
Feature: Split test smaller lesson content font size.
Because: * We are forcing larger font size on all users when it should be left as a choice. * Larger text accentuates and makes other things harder to get right - it had a part to play in the difficulty styling our links. This commit: * Experiment with using a baseline 16px font size. * Only 50% of users will see it so we can monitor feedback. * The best feedback for this is probably no feedback. The right font size is table stakes, if no one notices the change it will likely be a good thing.
- Loading branch information
1 parent
6d888f4
commit 7cc8207
Showing
3 changed files
with
11 additions
and
4 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,3 +1,3 @@ | ||
<div class="lesson-content max-w-prose mx-auto prose md:prose-lg prose-gray prose-a:text-blue-800 visited:prose-a:text-purple-800 prose-code:bg-gray-100 prose-code:p-1 prose-code:font-normal dark:prose-a:text-blue-300 dark:visited:prose-a:text-purple-300 dark:prose-code:bg-gray-700/40 prose-code:rounded-md break-words line-numbers dark:prose-invert dark:antialiased prose-pre:rounded-xl prose-pre:bg-slate-800 prose-pre:shadow-lg dark:prose-pre:bg-slate-800/70 dark:prose-pre:shadow-none dark:prose-pre:ring-1 dark:prose-pre:ring-slate-300/10 <%= classes %>" data-controller="syntax-highlighting diagramming" <%= html_data_attributes_for(data_attributes) %>> | ||
<div class="lesson-content max-w-prose mx-auto prose <%= font_size %> prose-gray prose-a:text-blue-800 visited:prose-a:text-purple-800 prose-code:bg-gray-100 prose-code:p-1 prose-code:font-normal dark:prose-a:text-blue-300 dark:visited:prose-a:text-purple-300 dark:prose-code:bg-gray-700/40 prose-code:rounded-md break-words line-numbers dark:prose-invert dark:antialiased prose-pre:rounded-xl prose-pre:bg-slate-800 prose-pre:shadow-lg dark:prose-pre:bg-slate-800/70 dark:prose-pre:shadow-none dark:prose-pre:ring-1 dark:prose-pre:ring-slate-300/10 <%= classes %>" data-controller="syntax-highlighting diagramming" <%= html_data_attributes_for(data_attributes) %>> | ||
<%= content %> | ||
</div> |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,10 +1,17 @@ | ||
class ContentContainerComponent < ApplicationComponent | ||
def initialize(classes: '', data_attributes: {}) | ||
def initialize(classes: '', data_attributes: {}, current_user: nil) | ||
@classes = classes | ||
@data_attributes = data_attributes | ||
@current_user = current_user | ||
end | ||
|
||
private | ||
|
||
attr_reader :classes, :data_attributes | ||
attr_reader :classes, :data_attributes, :current_user | ||
|
||
def font_size | ||
return '' if current_user && Feature.enabled?(:lesson_content_font_size, current_user) | ||
|
||
'md:prose-lg' | ||
end | ||
end |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters