Skip to content
This repository has been archived by the owner on Jun 26, 2023. It is now read-only.

Dialog cuts buttons #6

Open
alexolog opened this issue Jun 12, 2014 · 18 comments
Open

Dialog cuts buttons #6

alexolog opened this issue Jun 12, 2014 · 18 comments

Comments

@alexolog
Copy link

See http://tinypic.com/view.php?pic=2i9361e&s=8
Running FF30 with Classic Theme Restorer

@spasche
Copy link
Owner

spasche commented Jun 18, 2014

I tried to reproduce on Windows 8.1 with Firefox 30 by installing Open in Browser, FlashGot and Classic Theme Restorer, but the dialog looks fine for me:
capture

What's your operating system?

@alexolog
Copy link
Author

On 2 different machines: WinXP/SP3 and Win7

Perhaps there's a conflict with OpenDownload2?
https://addons.mozilla.org/en-US/firefox/addon/opendownload-10902/

@dertuxmalwieder
Copy link

I - as the author of OpenDownload2 - actually doubt that my extension fails, the only cases where there were major layout issues were always related to Open In Browser. Does your extension maybe resize the dialog?

@spasche
Copy link
Owner

spasche commented Jun 19, 2014

The extension doesn't resize the dialog. I couldn't reproduce on a Windows 7 vm, with a few extensions loaded. I can suggest that you try with a new clean profile, and add your extensions one by one until you see the problem happen. Then you can determine which extension is responsible.

capture

@alexolog
Copy link
Author

Try with an executable to actually trigger the [Run] button

@spasche
Copy link
Owner

spasche commented Jun 20, 2014

Ok, I can reproduce if I only keep FlashGot and Open in Browser. capture

Apparently the dialog adapts to the content, but sometimes it is not tall enough. FlashGot doesn't seem to set the height explicitly. I don't know why the dialog isn't auto-sized properly, maybe that's because some elements are added dynamically after the dialog is rendered.

Who is responsible and what the fix should be? Not sure about that. If someone has a good idea and a patch or pull request that's welcome ;-).

@dertuxmalwieder
Copy link

Maybe check position/size of the "OK" button and compare them to the dimensions of the window?

@spasche
Copy link
Owner

spasche commented Jun 20, 2014

Yeah, that could be an option. Care should be take to do the resizing after all other extensions have populated the dialog (using a timeout with some polling for instance).

Honestly I don't think I'll have time to implement this in the near future (and as I'm not using FlashGot that's not high on my todo list). If someone wants to tackle this, I'm happy to give directions.

@dertuxmalwieder
Copy link

I'm busy investigating about OD² 4.0's missing SeaMonkey compatibility, so I can't really help. :/

@spasche
Copy link
Owner

spasche commented Jun 20, 2014

Sure, no problem. I wasn't implying that you should implement it ;-)

@alexolog
Copy link
Author

Maybe talk to FlashGot's author?

@alexolog
Copy link
Author

alexolog commented Mar 4, 2015

Is there some progress on this?

@dertuxmalwieder
Copy link

I haven't fiddled with it yet. Should I (have to)?

@alexolog
Copy link
Author

Please do!

@dertuxmalwieder
Copy link

Damn :(

@alexolog
Copy link
Author

@dertuxmalwieder
Copy link

Appreciated!

@alexolog
Copy link
Author

The cookie is getting stale...

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants