URL: https://demo.optimoiot.it/dashboard/allarmi/config

Browser/Version: Firefox 80.0.1

Operating System: Windows 10 Tested Another Browser: Yes Edge

What seems to be the trouble?(Required)

• [ ] Desktop site instead of mobile site
• [ ] Mobile site is not usable
• [ ] Video doesn't play
• [x] Layout is messed up
• [ ] Text is not visible
• [ ] Something else (Add details below)

Steps to Reproduce

1. Navigate to:
2. Click three times over the header of column "Nome" so that the column is ordered by descending names

Expected Behavior: All the columns should be visible

Actual Behavior: The extremely long name shifts the column to the right and so the last ones can't be accessed anymore. No horizontal scrollbar is present.

With Edge 85.0.564.51 the names overlap (and this is another issue), but the column alignment is as expected.

Screenshot

Thank you for your investigation. The graph has been changed and now the zoom works in firefox too.

<!-- @browser: Firefox 80.0 --> <!-- @ua_header: Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:80.0) Gecko/20100101 Firefox/80.0 --> <!-- @reported_with: addon-reporter-firefox -->

URL: https://demo.optimoiot.it/dashboard/main

Browser / Version: Firefox 80.0 Operating System: Windows 10 Tested Another Browser: Yes Edge

Problem type: Design is broken Description: Items are overlapped Steps to Reproduce: When you expand the item "Allarmi" on the left side, the items overlap the item at the bottom of the page <details> <summary>View the screenshot</summary> </details>

<details> <summary>Browser Configuration</summary> <ul> <li>None</li> </ul> </details>

From webcompat.com with ❤️

<!-- @browser: Firefox 80.0 --> <!-- @ua_header: Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:80.0) Gecko/20100101 Firefox/80.0 --> <!-- @reported_with: addon-reporter-firefox -->

URL: https://demo.optimoiot.it/dashboard/main

Browser / Version: Firefox 80.0 Operating System: Windows 10 Tested Another Browser: Yes Edge

Problem type: Site is not usable Description: Buttons or links not working Steps to Reproduce: Move the mouse over the chart "Temperature". The mouse icone chnages to hand (and this is ok). This should enable the zoom functionality using the mouse wheel, but this doesn't happen. If you scroll the wheel, the page is scrolled up and down. Maybe the chart doesn't take the focus? <details> <summary>View the screenshot</summary> </details>

<details> <summary>Browser Configuration</summary> <ul> <li>None</li> </ul> </details>

From webcompat.com with ❤️

I'm not sure I understood completely the idea, but as far as I understand it sounds interesting. You refer to a library of C++ high-level components, I think. Right?

@JonatanAntoni one more yeas passed by and no news on this topic. You wrote

I will do my very best to support anybodies initiative.

but I think you should do something encouraging this kind of initiative (i.e. promote a contest, ask for volunteers, give publicity to this task ...). Otherwise I don't think that someone would self-assign this kind of activity

I understood. Now it should be ok

AppVeyor takes a lot of time to execute and so enabling 'Rolling builds' can help a lot. See here. Are 'Rolling builds' enabled for rapidjson project?

While writing the unit test I found out one important thing that must be clarified. What is the purpose for clzll() function? Counting the number of leading zeroes? If this is the case, the original naive fallback was right, and so the https://github.com/escherstair/rapidjson/blob/58e29648561e47f6fc11ee2c2bbf8628ffc46f57/include/rapidjson/internal/clzll.h#L51 But the _MSC_VERimplementation is not, because _BitScanReverse() returns the index on the first 1 (starting the search from MSB, but the index for LSB is 0).

Let me know, please

Would you mind adding an unit test (including the boundary cases) for this function?

I can try to do my best, but I need time to understand how to use gtest.

Can you confirm that 0 is not an acceptable value for as an input to clzll(uint64_t x)? I see https://github.com/Tencent/rapidjson/blob/6364c8e5ab1b9b5df01df25ff102d1c48aa0d389/include/rapidjson/internal/clzll.h#L35

I followed your suggestion to fallback to naive function. But I think that the naive function needs a couple of fixes (i.e. r variable initialization and post-decrementing it)

I rebased over v1.x branch, squashed the commits and updated the messages

I've just done PR #1760

I rebased but I have problems in squashing and changing commit labels

File clzll.h includes <intrin.h> if the compiler is MSC

#if defined(_MSC_VER)
#include <intrin.h>
#if defined(_WIN64)
#pragma intrinsic(_BitScanReverse64)
#else
#pragma intrinsic(_BitScanReverse)
#endif
#endif

But if the target is CE6 with Visual Studio 2008, intrin.h is not available and the application can't be compiled rapidjson\internal/clzll.h(21) : fatal error C1083: Cannot open include file: 'intrin.h': No such file or directory

As far as I understand, intrin.h is necessary to call _BitScanReverse()

How can be fixed this include so that support for CE6 is not dropped (rapidjson 1.1.0 fully supports CE6)?