Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

FLUID-4698: setting font-size on html #497

Merged
merged 5 commits into from Apr 25, 2014

Conversation

jobara
Copy link
Member

@jobara jobara commented Apr 23, 2014

Modified the textSize enactor to set the font-size on the root element (html) instead of the container (body). This will allow support of rem units. Another change with this was to set the value in px instead of ems. This is necessary since the root element has no parent for which to base it's value on. The unit tests have been updated.

http://issues.fluidproject.org/browse/FLUID-4698

Modified the textSize enactor to set the font-size on the root element (html) instead of the container (body). This will allow support of rem units. Another change with this was to set the value in px instead of ems. This is necessary since the root element has no parent for which to base it's value on. The unit tests have been updated.
@cindyli
Copy link
Member

cindyli commented Apr 24, 2014

There's an issue with this pull request:

  1. Adjust src/demos/uiOptions/index.html to set the root element with font-size: 2em (or 32px);
  2. Open src/demos/uiOptions/index.html, the initial font size is set correctly with the big fonts;
  3. Simply open UIO panel without adjusting any setting, the font on the page shrinks to the default browser font size.

@cindyli
Copy link
Member

cindyli commented Apr 24, 2014

@jobara, another issue is the change on the font size is not applied to the UIO panel when you close and re-open it. The change is applied to the main page correctly.

Fixed an issue where the text size would always be set on the parent document. This was causing issues with the separated panel which uses an iframe for the controls and has its own enactors. Also made another small modification that ensures the multiplier used is always a legitimate value.
@cindyli
Copy link
Member

cindyli commented Apr 24, 2014

Enactors-test.html and UIEnhancer-test.html fail in IE (tried with IE8 & 9) but not other browsers.

In the master branch, Enactors-test.html already fails with UIEnhancer-test.html passes. Perhaps an opportunity to fix Enactors test too.

@jobara
Copy link
Member Author

jobara commented Apr 25, 2014

@cindyli I tested in IE 8, 9, 10, & 11.

I've filed a new jira for the enactor failure for lineSpace http://issues.fluidproject.org/browse/FLUID-5320

I wasn't able to reproduce the failing tests for UIEnhancer.

In IE 8 I also had a failure for the builder tests but they seemed to be related to jqUnit not being created properly. I'm not sure why this is happening though.

@cindyli
Copy link
Member

cindyli commented Apr 25, 2014

We can address the builder failure in IE8 in another jira since it happens for the master branch too. I've filed a jira for it: http://issues.fluidproject.org/browse/FLUID-5323

In terms of the UIEnhancer failure, I cannot reproduce it with sauceLabs environment too. It might be something specific in my local VM.

@cindyli cindyli merged commit f6b8ca0 into fluid-project:master Apr 25, 2014
@cindyli
Copy link
Member

cindyli commented Apr 25, 2014

Merged into the project repo @ c7c61c4

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants