search

topcoderinc / TC-redis-manager

5 stars 15 forks source link

Clear all history text should change #338

Open codejamtc opened 5 years ago

codejamtc commented 5 years ago

image

  1. Deploy the App and run the application
  2. Select a server
  3. Put the cursor on the history icon on the redis command window

Actual: Clear all history

Expected: Clear All History

Browser(s): Firefox 64.0.2 (Windows)

codejamtc commented 5 years ago

Same issue on the Previous page and Next page

image

image

kinfkong commented 5 years ago

I fixed for the "Clear All History". but the "Next page"/"Previous page" are from the library. if we have to fix this, we have to add a custom class: https://stackoverflow.com/questions/46869616/how-to-use-matpaginatorintl/46883528

I think this is a little overkill. tell me if it is require to fix the next/previous page labels. If yes, I will continue the fixes.