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
PR - Ticket 49706 - Finish UI patternfly convertions #2766
Comments
Comment from vashirov (@vashirov) at 2018-05-23 13:51:49 Some tables use default serif font in their headers: Buttons at the top (instance selection and actions) also are rendered using custom style instead of PF. The rest looks good. |
Comment from mreynolds (@mreynolds389) at 2018-05-23 14:11:31
Well these are bootstrap tables as recommended in PF docs: http://www.patternfly.org/pattern-library/widgets/#tables Datatables are too busy for these IMO, but I could switch them to DT's. Perhaps I can strip off some of the datatable features...
This was intentional, I kind of want to keep them as is. It gives the banner "a little" character.
|
Comment from vashirov (@vashirov) at 2018-05-23 14:30:40 Here's how tables are rendered from the link you gave: https://vashirov.fedorapeople.org/share/pf_table.png And buttons for me look like this: https://vashirov.fedorapeople.org/share/buttons.png And I've just noticed, that we're still trying to load the fonts that we removed (woff2 and ttf files), they are mentioned in patternfly.css that is shipped with 389-console. |
Comment from mreynolds (@mreynolds389) at 2018-05-23 14:59:19
Weird! For me the tables look the same, but your png files are definitely different. What browser are you using? I was using chrome. I also did not add any styling/fonts to the tables btw.
haha! Perhaps we should
Maybe I'll add those font files back. I'd prefer to not edit the PF css |
Comment from vashirov (@vashirov) at 2018-05-23 15:43:40
These were taken in Chrome, but in Firefox I see similar issues. Looks like table header font falls back to sans-serif font if Open Sans, Helvetica or Arial are not found. I've added fonts back and it fixed the problem. |
Comment from mreynolds (@mreynolds389) at 2018-05-23 20:46:32 rebased onto 164a959 |
Comment from mreynolds (@mreynolds389) at 2018-05-23 20:48:04 I changed the buttons/dropdowns in the banner, and added the font files back that the browser was complaining about |
Comment from mreynolds (@mreynolds389) at 2018-05-23 20:48:37 Pull-Request has been merged by mreynolds389 |
Patch |
Cloned from Pagure Pull-Request: https://pagure.io/389-ds-base/pull-request/49707
Description: A few modals were not converted to PF, and other minor issues
with JS and updating tables
Resolves: #2765
Reviewed by: ?
The text was updated successfully, but these errors were encountered: