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

Buttons lack text when using the "valo" theme with the TouchKit widgetset #15

Open
steinarb opened this issue May 14, 2018 · 3 comments · May be fixed by #16
Open

Buttons lack text when using the "valo" theme with the TouchKit widgetset #15

steinarb opened this issue May 14, 2018 · 3 comments · May be fixed by #16

Comments

@steinarb
Copy link

See the thread Text missing from buttons in valo theme. This has been an issue back with Vaadin 7.x and TouchKit 4.x as well.

However with Vaadin 8.x and TouchKit 5.x the only theme where the grids (that have replaced tables) look properly, is valo. And with valo the buttons don't display text.

So it would be nice to have this problem fixed.

@steinarb
Copy link
Author

As this posting on the thread shows, the offending CSS style, is this one:

.v-button-wrap {
    display: block;
    padding: 0 10px;
    white-space: nowrap;
    text-overflow: ellipsis;
    overflow: hidden;
}

Removing the "display: block" from this rule makes button text reappear when using valo with the TouchKit widgetset.

But I don't know if this is the correct fix.

steinarb added a commit to steinarb/touchkit that referenced this issue May 14, 2018
@mstahv
Copy link

mstahv commented Jun 7, 2018

I think this relates to the fact the touchkit theme is designed to be used as such, without valo theme, so there are collisions.

There would basically be two options:

  • extend touchkit theme so that it supports Grid. I'd say this is the best way to solve the issue
  • more rules (and possibly stronger selectors) to the touchkit theme so that collisions with Valo theme are handled properly

What do you think?

@steinarb
Copy link
Author

steinarb commented Jun 9, 2018

From my point of view getting workable grid formatting would be the best way to proceed in the short term. Then I can use the TouchKit theme and have stuff working like it did with Vaadin7/TouchKit4

Handling collisions with the valo theme better also sounds like a good thing. But from my point of view this can wait until the Vaadin8/TouchKit5 version works at least as well as the Vaadin7/TouchKit4 version did.

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 a pull request may close this issue.

2 participants