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

page-link-prev page-link-next font size error #903

Open
ShamoX opened this issue Nov 14, 2018 · 2 comments
Open

page-link-prev page-link-next font size error #903

ShamoX opened this issue Nov 14, 2018 · 2 comments

Comments

@ShamoX
Copy link

ShamoX commented Nov 14, 2018

I use this module in several places and I have a visual bug I can't understand.
I use it (ng2-smart-table 1.3.5) along side with:

  • angular 6.1.7
  • ng-bootstrap 3.3.1 (bootstrap 4.1.3)
  • ng2-tree 2.0.0-rc.11
  • ... Do you need all the list ?

I have on Safari and Chrome (both on MacOS) a little incoherency on the definition of the page-link-prev and page-link-next font size here.

This make the '<' and '>' smaller that the rest of the pager line. See the screenshot:
with font-size

This is corrected by removing the line of css that set the font-size for those links:
without font-size

I'm I the only one to get this glitch ? Maybe I'm not using it correctly somewhere or there is some incompatibility with my configuration ?

@An3ramirez
Copy link

An3ramirez commented Dec 2, 2018

hello friend I have the same problem

I fixed it in a slightly drastic way modifies the scss of the plugin modifies this file
proyect/node_modules/ng2-smart-table/components/pager/pager.component.scss

in line 21
.page-link-next, .page-link-prev { font-size: 10px; }

modifi to

.page-link-next, .page-link-prev { font-size: 0.85rem; }

@prathap0611
Copy link

I too faced this issue, I guess it js not a good idea to change the scss file under node_modules.
As a work around , you can overwrite the property in your app. The below code fixed the issue for me,

:host {
  /deep/ {
    .page-link-prev, .page-link-next {
       font-size: 0.875rem !important;
      }
  }
} 

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

No branches or pull requests

3 participants