Skip to content

Conversation

@nickdesaulniers
Copy link

review? @cvan

core/vr-text.js Outdated
Copy link
Member

Choose a reason for hiding this comment

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

Can people use styles on the element to override this?

Copy link
Contributor

Choose a reason for hiding this comment

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

yeah, I would add these styles to our stylesheet, so the styles can get overridden without doing !important by the user

@cvan
Copy link
Contributor

cvan commented Sep 21, 2015

looks good! file-up issues would be great 👍

cvan added a commit that referenced this pull request Sep 21, 2015
@cvan cvan merged commit 8a9ad4f into master Sep 21, 2015
@nickdesaulniers nickdesaulniers deleted the text branch September 21, 2015 19:32
cvan added a commit that referenced this pull request Nov 30, 2015
@nickdesaulniers nickdesaulniers removed their assignment Dec 7, 2015
cvan added a commit that referenced this pull request Dec 9, 2015
cemkod pushed a commit to cemkod/aframe that referenced this pull request Dec 23, 2015
Fixed nested iframe web element find
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

4 participants