opened
vasild
username_0
<!--

Got a question?
===
The issue list of this repo is **exclusively** for bug reports and feature requests. For simple questions, please use the following resources:

- Read the docs: https://v1.quasar-framework.org
- For simple/quick questions ask on the Discord chat room: http://chat.quasar-framework.org/
- For complex questions or requiring help, ask on the forum: https://forum.quasar-framework.org/

Reporting a bug?
================
- Are you sure it isn't already reported? Do a search first! It may have already been answered or even fixed in the development branch (`dev`).

- If you are reporting a bug for Quasar v1.0+, then please try to include a forked JsFiddle/Codepen: https://jsfiddle.net/username_2/rmaodk0f or https://codepen.io/username_2/pen/VgQbdx

- If you are reporting a bug for Quasar v0.15+, then please try to include a forked JsFiddle/Codepen from one of these: https://jsfiddle.net/username_2/waugrryy/ (jsFiddle - Material theme) or https://jsfiddle.net/username_2/7gu065yg/ (jsFiddle - iOS theme) or https://codepen.io/username_2/pen/KQRZJg (Codepen - Material theme) or https://codepen.io/username_2/pen/paVpBN (Codepen - iOS theme)

- Check if the issue is reproducible with the latest stable version of Quasar. If you are using a pre-release, please indicate the specific version you are using.

- It is **required** that you clearly describe the steps necessary to reproduce the issue you are running into. Issues with no clear repro steps will not be triaged. If an issue labeled "need repro" receives no further input from the issue author for more than 5 days, it will be closed.

- If your issue is resolved but still open, don’t hesitate to close it. In case you found a solution by yourself, it could be helpful to explain how you fixed it.

Have a feature suggestion/request?
=======================
Remove the template from below and provide thoughtful commentary *and code samples* on what this feature means for your product. What will it allow you to do that you can't do today? How will it make current workarounds straightforward? What potential bugs and edge cases does it help to avoid? etc. Please keep it product-centric.
-->

<!-- BUG REPORT TEMPLATE -->
### Software version

<!-- You can use the `quasar info` command and copy paste it here -->
Quasar: 1.0.0-beta.23
@quasar/app (v1+ only): v1.0.0-beta.26
OS: FreeBSD 12
Node: 12.2.0
NPM: 6.9.0
Browsers: Firefox 67.0
iOS:
Android:
Any other software related to your bug:

### JsFiddle
<!-- v1 only: Fork this https://jsfiddle.net/username_2/rmaodk0f or https://codepen.io/username_2/pen/VgQbdx and fill it in here -->
No need for jsfiddle, as the bug is observed also on the docs page: https://v1.quasar-framework.org/vue-components/table#Usage

<!-- v0.15-17: fork this https://jsfiddle.net/username_2/waugrryy/ and fill it in here -->

### What did you get as the error?
Some tables have `Records per page: 5` at the bottom. Nothing changes visually when `5` is clicked. The console shows:
```
InternalError: "too much recursion"
    focus https://v1.quasar-framework.org/js/vendor.5793ed61.js:24
    showPopup https://v1.quasar-framework.org/js/vendor.5793ed61.js:24
    focus https://v1.quasar-framework.org/js/vendor.5793ed61.js:24
    showPopup https://v1.quasar-framework.org/js/vendor.5793ed61.js:24
...
```

### What were you expecting?
A menu should be displayed showing various options for `Records per page`, e.g. 5, 10, 20.

### What steps did you take, to get the error?
Just open https://v1.quasar-framework.org/vue-components/table#Usage and try to click `5` below some of the tables, next to `Records per page`

This is a regression in 1.0.0-beta.23 and the bug does not exist in 1.0.0-beta.22.
I confirm that commit f6ad9077 is the one that broke it - `f6ad9077~` is ok, `f6ad9077` is broken.
Table rows-per-page menu broken
issue
created
vasild
username_0
The bug is not fixed in the latest `dev` branch - e7c0aba4 still exhibits it.
comment
created
pdanpdan
username_1
Can you please tell us what user agent you're using on the browser?
comment
closed
rstoenescu
username_2
issue
created
rstoenescu
username_2
@username_0 Fix will be available in v1-rc.1. Can you confirm this?
And also, please tell us the user agent of your browser. Thank you.
comment
created
vasild
username_0
Thanks for the quick response.

I confirm that the bug is fixed in the latest `dev` branch (02143345). I guess fixed by f1abe907.

My UA is `Mozilla/5.0 (X11; FreeBSD amd64; rv:67.0) Gecko/20100101 Firefox/67.0`. It is not anonymized, but the previous code in `Platform.js` was a bit lame - assuming that if the OS is not "Windows, Linux or Mac" then it does not exist, kind of.

Thanks!
comment