leetcode-format-chrome-extension icon indicating copy to clipboard operation
leetcode-format-chrome-extension copied to clipboard
verified publisher icon madhur

Not Working

Open SumitNub12 opened this issue 2 years ago • 12 comments

I installed the extension but format button not showing in the UI

SumitNub12 avatar Dec 21 '23 16:12 SumitNub12

@SumitNub12 Can you post screenshot and also if there are any error in console window?

madhur avatar Dec 21 '23 17:12 madhur

The Format button stopped showing after I updated my chrome to Version 120.0.6099.109 (Official Build) (64-bit). I am not seeing any error in console

shubhamsharma04 avatar Dec 23 '23 08:12 shubhamsharma04

On Chrome Version 120.0.6099.109 (Official Build) (x86_64), extension version: 1.8

Doesn't work in Dynamic Layout

Screen Shot 2023-12-23 at 6 46 14 PM

Works in Split (old) layout

Screen Shot 2023-12-23 at 6 47 08 PM

Tested on page: https://leetcode.com/problems/sum-root-to-leaf-numbers/description/

SkandaB avatar Dec 24 '23 03:12 SkandaB

Possibly related to (from Dev console)

div.flex.flex-nowrap.items-center

Source code looking for

div.flex div.items-center div.mr-auto

not matching?

On Chrome Version 120.0.6099.109 (Official Build) (x86_64), extension version: 1.8

Doesn't work in Dynamic Layout

Screen Shot 2023-12-23 at 6 46 14 PM

Works in Split (old) layout

Screen Shot 2023-12-23 at 6 47 08 PM

Tested on page: https://leetcode.com/problems/sum-root-to-leaf-numbers/description/

SkandaB avatar Dec 24 '23 03:12 SkandaB

@SkandaB I have released the fix for this issue. This should be automatically released once its approved https://github.com/madhur/leetcode-format-chrome-extension/pull/27

madhur avatar Dec 24 '23 10:12 madhur

@SkandaB @shubhamsharma04 The new version has been released. Pls check if this fixes the issue.

madhur avatar Dec 25 '23 12:12 madhur

Checked with v 1.9 of the extension Not working

(Tried these options)

  1. Delete cache and cookies
  2. Disable and enable extension
  3. Log out and log in again
  4. Change to split layout and back to dynamic again

None of these worked

SkandaB avatar Dec 25 '23 19:12 SkandaB

@madhur , The Format has moved to Testcase section Screen Shot 2023-12-25 at 2 01 18 PM

Also script.js#L29 is incorrect

Should be

if (document.getElementById('format-button') !== null)

Will help in debugging.

SkandaB avatar Dec 25 '23 22:12 SkandaB

@SkandaB @shubhamsharma04 The new version has been released. Pls check if this fixes the issue.

Its working for me

shubhamsharma04 avatar Dec 26 '23 06:12 shubhamsharma04

@madhur , The Format has moved to Testcase section Screen Shot 2023-12-25 at 2 01 18 PM

Also script.js#L29 is incorrect

Should be

if (document.getElementById('format-button') !== null)

Will help in debugging.

@SkandaB Do you have a premium account? It seems to be having an issue because of presence of debugger, which normal accounts do not have.

Unfortunately, I do not have premium account , so I am unable to reproduce.

madhur avatar Dec 26 '23 16:12 madhur

Yes, premium account.

No worries, thanks for the attempt. Appreciate it

On Tue, Dec 26, 2023, 8:00 AM Madhur Ahuja @.***> wrote:

@madhur https://github.com/madhur , The Format has moved to Testcase section [image: Screen Shot 2023-12-25 at 2 01 18 PM] https://private-user-images.githubusercontent.com/5369621/292780495-15899fb7-c6bc-4a9f-a376-08e5a131897a.png?jwt=eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9.eyJpc3MiOiJnaXRodWIuY29tIiwiYXVkIjoicmF3LmdpdGh1YnVzZXJjb250ZW50LmNvbSIsImtleSI6ImtleTEiLCJleHAiOjE3MDM2MDY2NzMsIm5iZiI6MTcwMzYwNjM3MywicGF0aCI6Ii81MzY5NjIxLzI5Mjc4MDQ5NS0xNTg5OWZiNy1jNmJjLTRhOWYtYTM3Ni0wOGU1YTEzMTg5N2EucG5nP1gtQW16LUFsZ29yaXRobT1BV1M0LUhNQUMtU0hBMjU2JlgtQW16LUNyZWRlbnRpYWw9QUtJQUlXTkpZQVg0Q1NWRUg1M0ElMkYyMDIzMTIyNiUyRnVzLWVhc3QtMSUyRnMzJTJGYXdzNF9yZXF1ZXN0JlgtQW16LURhdGU9MjAyMzEyMjZUMTU1OTMzWiZYLUFtei1FeHBpcmVzPTMwMCZYLUFtei1TaWduYXR1cmU9M2E0Y2ExMGNmYjA4Y2Y2N2Y4ZDRlNmExODEyZjc2MmI5N2Q1ODJkNzAyMjM0MTA1M2I0N2E2NGVhZGJjZDE0MCZYLUFtei1TaWduZWRIZWFkZXJzPWhvc3QmYWN0b3JfaWQ9MCZrZXlfaWQ9MCZyZXBvX2lkPTAifQ.QEnV2KU3QPnJK85p4TlO-nVDnh0J4FubaWzOH3LvQNU

Also script.js#L29 https://github.com/madhur/leetcode-format-chrome-extension/blob/25dc76ad7472e52b84a4056a9f751456a1408194/script.js#L29 is incorrect

Should be

if (document.getElementById('format-button') !== null)

Will help in debugging.

@SkandaB https://github.com/SkandaB Do you have a premium account? It seems to be having an issue because of presence of debugger, which normal accounts do not have.

Unfortunately, I do not have premium account , so I am unable to reproduce.

— Reply to this email directly, view it on GitHub https://github.com/madhur/leetcode-format-chrome-extension/issues/26#issuecomment-1869633381, or unsubscribe https://github.com/notifications/unsubscribe-auth/ABI66FPBI2V7IWIKHE4HHWLYLLYCNAVCNFSM6AAAAABA6T3I76VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQNRZGYZTGMZYGE . You are receiving this because you were mentioned.Message ID: @.***>

SkandaB avatar Dec 26 '23 16:12 SkandaB

@madhur , The Format has moved to Testcase section Screen Shot 2023-12-25 at 2 01 18 PM Also script.js#L29 is incorrect Should be

if (document.getElementById('format-button') !== null)

Will help in debugging.

@SkandaB Do you have a premium account? It seems to be having an issue because of presence of debugger, which normal accounts do not have.

Unfortunately, I do not have premium account , so I am unable to reproduce.

Thanks for looking into this. FWIW, I have a premium account and it is working for me. The problem could be something else other than the debugger.

shubhamsharma04 avatar Dec 27 '23 03:12 shubhamsharma04