Skip to content
This repository was archived by the owner on Sep 6, 2021. It is now read-only.

Adding interfaces to change font size, font family, and line height. #8305

Closed
MiguelCastillo wants to merge 13 commits into from
Closed

Adding interfaces to change font size, font family, and line height. #8305

Show file tree
Hide file tree
Changes from 12 commits
Commits
Show all changes
13 commits
Select commit Hold shift + click to select a range
97d00ef
Adding interfaces to change font size, font family, and line height.
MiguelCastillo Jul 3, 2014
2e72136
Adding more display inline blocks so that line height renders correctly
MiguelCastillo Jul 3, 2014
57e33ad
Renamed fontSizeStyle preference to fontSize
MiguelCastillo Jul 3, 2014
4e463ec
Changed line height values to be floats instead of strings with units…
MiguelCastillo Jul 3, 2014
1b358c9
Refactored fonts manager to handle different permutations of opening/…
MiguelCastillo Jul 4, 2014
a84f934
added/removed a few empty lines to give the code a little breathing room
MiguelCastillo Jul 4, 2014
395dd1e
Removed preference change listeners. We should be using the API
MiguelCastillo Jul 4, 2014
0650b83
Adding init function to ensure values are loaded when appReady is called
MiguelCastillo Jul 8, 2014
09650b2
Added a more specific css rule for line height settings to avoid conf…
MiguelCastillo Jul 8, 2014
24177a0
Removed stale variables and jslint definitions
MiguelCastillo Jul 8, 2014
bad3395
Removed stale code in favor of initializing the font when the app is …
MiguelCastillo Jul 8, 2014
c464845
Removing em unit from line height DOM element
MiguelCastillo Jul 8, 2014
cd99a46
Merge master
MiguelCastillo Jul 15, 2014
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
12 changes: 11 additions & 1 deletion src/styles/brackets_codemirror_override.less
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -129,7 +129,17 @@
*/
pointer-events: none;
}


/**
* These classes are set to inline-block because they are spans and they need block dimension properties when
* when calculation height and width.
*/
.CodeMirror-searching,
.CodeMirror-matchingtag,
.CodeMirror-matchingbracket {
display: inline-block;
}
Copy link
Copy Markdown
Member

@peterflynn peterflynn Jul 14, 2014

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why is this required now when it wasn't before? I'm unsure what about making font size configurable would change something about this...

Copy link
Copy Markdown
Contributor Author

@MiguelCastillo MiguelCastillo Jul 14, 2014

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

So, when line height is changed some items like those don't properly scale because they are spans. To quickly test this behavior, try increasing the line height in the current themes to something like 1.8 and place the cursor on a matching bracket. Or select a word and press CMD+F to find matching search.


.CodeMirror-linewidget {
/* Re-enable pointer events for line widget. Pointer events are disabled for "CodeMirror-lines", which is the
* parent of line widgets, so they need to be explicitly re-enabled here in order for selection to work. */
Expand Down
Loading
⚡