Closed (fixed)
Project:
Drupal core
Version:
8.0.x-dev
Component:
ckeditor.module
Priority:
Normal
Category:
Bug report
Assigned:
Issue tags:
Reporter:
Created:
3 Oct 2014 at 15:03 UTC
Updated:
1 Feb 2015 at 23:54 UTC
Jump to comment: Most recent, Most recent file

Comments
Comment #1
wim leersThanks, great point :)
Comment #2
rpayanmTrying...
Comment #3
rpayanmWith 14px.

Comment #4
rpayanmWith 15px.

Comment #5
rpayanmWith 16px.

Comment #9
xlin commentedI updated the font-size to 14px. Please review.
Comment #11
xlin commentedSecond try.
Comment #12
xlin commentedRe-upload with correct comment #.
Comment #14
Palashvijay4O commentedSubmitting a test patch .
Comment #15
Palashvijay4O commentedScreenshots
Comment #22
Palashvijay4O commentedThis seems little strange same patch passed on re-test .... Can anyone elaborate on this ?
Comment #23
rpayanm@Palashvijay4: you can't assigned the issues from other users, only you can assigned when no one has.
@Palashvijay4,@xq1003: For work an any issue check first that this is unassigned, if no two people working on the same thing, being able to work in another issue.
Greetings.
Comment #24
BarisW commentedThere is a comment in the CSS stating "A font-size of 16px prevents iOS from zooming."
So just changing the font-size from 16px to 15px doesn't seem like a good idea without finding out what this comment is about. If there's a workaround, we need to apply that too and update the comment.
Comment #25
rpayanm@BarisW I upload 3 patches without touch that line and for large screens I changed the font-size to 14px, 15px and 16px, in the comments #3, #4, #5 respectively, you try with those?
Comment #26
rpayanmComment #27
wim leersPerhaps we should look into http://makc3d.wordpress.com/2014/11/08/responsive-font-size-with-ckeditor/ and adopt something like that?
(Found it via http://ckeditor.com/blog/CKEditor-Weekly-for-November-17-2014.)
Comment #28
zealfire commented@wim leers and @Baris W, i think the comment which is present in css file of ckeditor with respect to zooming in iOS is related to comfortable reading size which i found at his link :http://uxcellence.com/2014/01/15/quick-fix-increase-font-size-to-16px-to... ,so there is most probably no need of any change in this section.But i have made changes in body section by changing font size of 12px to 15px.Am i going in the right direction ?
Comment #29
Palashvijay4O commented@zealfire you should reset your previous changes to the codebase and then create a patch for a new issue . Alternatively you can create different branches for different issue . Hope this helps . :)
Comment #31
zealfire commenteduploading new patch.@wim leers and @Baris W, i think the comment which is present in css file of ckeditor with respect to zooming in iOS is related to comfortable reading size which i found at his link :http://uxcellence.com/2014/01/15/quick-fix-increase-font-size-to-16px-to... ,so there is most probably no need of any change in this section.But i have made changes in body section by changing font size of 12px to 15px.Am i going in the right direction ?
Comment #32
katetak commentedI tested the patch in #31 and it does change the font size in the window. It's at 15px on standard screen, and increases to 16px on smaller screen.
Comment #33
arunkumarkTested and Reviewed in Safari, Opera, IE 11, Chrome and Firefox on Windows.


Attached Screen shot for Safari:
Attached Screen shot for Chrome:
Its works Fine.
Comment #34
lewisnymanNice work, I guess it should be pulling in the stylesheet from the active theme? Sounds like a different issue though.
Comment #35
yoroy commentedThanks for reviewing and screenshots, rtbc indeed.
Comment #36
alexpottCommitted 02d58f6 and pushed to 8.0.x. Thanks!