Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

ask for " button maximize " #366

Open
bgbruno opened this issue Aug 1, 2023 · 15 comments
Open

ask for " button maximize " #366

bgbruno opened this issue Aug 1, 2023 · 15 comments

Comments

@bgbruno
Copy link

bgbruno commented Aug 1, 2023

issue

panel after open is on bottom
if I need more space on mobile I have to resizing everytime

solution

add button " maximize , minimize " on right top
left from button " close " from #365

@bgbruno bgbruno changed the title ask for " close maximize " ask for " button maximize " Aug 1, 2023
@adityarawat1223
Copy link

can I work on this?

@bgbruno
Copy link
Author

bgbruno commented Aug 10, 2023

@adityarawat1223 I think this is point of opensource - everyone can do that and then - pull request - what wait for repo admin to merge

Q) or is here some other workflow ?

@JingMatrix
Copy link

This is already implemented. You can click on the icon to toggle maximize / minimize.

@bgbruno
Copy link
Author

bgbruno commented Aug 13, 2023

This is already implemented

I use version 3.0.0 and I don't see it there

You can click on the icon to toggle maximize / minimize.

Q) pls can you show screenshot @JingMatrix ?

@JingMatrix
Copy link

Look at the demo in the README:

Click on the icon in the bottom right corner. There are six of them shown in the above image.

@bgbruno
Copy link
Author

bgbruno commented Aug 13, 2023

still not sure what icons you mean

and I think it just missunderstanding - so I draw it 🙂

this is what missing me there

Screenshot 2023-08-13 at 17 44 40

@JingMatrix
Copy link

The icon in the bottom right of the img I gave, with shape similar to ⚙.

@bgbruno
Copy link
Author

bgbruno commented Aug 13, 2023

this is exactly why I open this ticket 🙂
because of this " panel float button "

at first - is still shown after panel is opened
at second - then they get in the way if there is another data
and at third - missing possibility maximize panel for comfortable developing if FE is not needed

my propose solution is that 3 standard icons for " minimize , maximize , close "
and after close then show your " float icon " for open " dev tools panel "

I hope this time I use better description 😀

@JingMatrix
Copy link

I disagree with your suggestion of introducing more icons.

  1. Of course, the icon should be shown all the time, so users can toggle it to hide all panels.
  2. Regarding removing this icon, run eruda.destroy().

Hence, I would suggest closing this issue and also #365

@bgbruno
Copy link
Author

bgbruno commented Aug 13, 2023

I disagree with your suggestion of introducing more icons.

these icons will bring more control to display panel where developer needs

Of course, the icon should be shown all the time, so users can toggle it to hide all panels.

yes - close feature will be visible all the time - by standard way with " cross icon - X " in " top right " side on panel

Regarding removing this icon, run eruda.destroy()

sure but I cannot add another button into panel so I can achieve standard way of panels

Hence, I would suggest closing this issue

if you decide it from only your 1 man perspective - yours - I do nothing about it

and also #365

ah yes - that ticket #365 is about this

Screenshot 2023-08-13 at 18 22 30

and this ticket #366 is about this

Screenshot 2023-08-13 at 18 22 39

and both can be handeled with those 3 icons like standard window control buttons

Screenshot 2023-08-13 at 17 44 40

@JingMatrix
Copy link

JingMatrix commented Aug 13, 2023

My main arguments is that three more icons is too crowed for most mobile devices.

Look at DevTools front-end shown on mobiles, thers is no x icon.
screenshot_2023-08-13T18:37:09

If the author decides to introducing more icons for wider screens, then it is better to hide them for other normal mobile devices.
I feel comfortable when eruda is similar to the above (actually, it is the official one) front-end design.

@bgbruno
Copy link
Author

bgbruno commented Aug 13, 2023

three more icons is too crowed for most mobile devices.

now I understand you more - and I agree of course - for mobile

Look at DevTools front-end shown on mobiles, thers is no x icon.

not sure how you opened it - because I don't see it in my #chrome - #ios or #android
BUT even so - how you want it hide then if there is no icon ?
so when the panel shows an "x" , everyone knows how to close any panel

more icons for wider screens ... hiden them for ... mobile devices

yes I agree - this is the right way

>>

by my proposal universal button " x " should be displayed for all screens - and all the time

at least like possibility to set buttons - like we can set " tabs " with param " tool "
https://github.com/liriliri/eruda/blob/master/doc/API.md#init

eruda.init({
	controlButtons : {
                shrinked : [ 'minimize' , 'maximize' ] ,
                default : [ 'close' ]
        }
});

@JingMatrix
Copy link

To answer to your questions, I open DevTools on Android using my own project ChromeXt. DevTools is opened in a new tab, so I close it by closing the tab.

This comment might be off-topic, feel free to hide it.

@bgbruno
Copy link
Author

bgbruno commented Aug 13, 2023

I open DevTools on Android using my own project ChromeXt

so this is not default functionality

DevTools is opened in a new tab, so I close it by closing the tab

with button " x " 😉 - otherwise you have to create some button to do that

This comment might be off-topic, feel free to hide it.

I think it is important to understand this theme

@JingMatrix
Copy link

I implemented some hooks to move panels around the button, so that the panels are moved (vertically) when toggling to show them after dragging the button.
Here is the demo:
screenshot_2023-08-15T23:57:41

Here are the raleted lines of this hook:
https://github.com/JingMatrix/ChromeXt/blob/f41f2f78ab3f8f33c554bb17dc4cdcdb4ddb8f30/app/src/main/assets/eruda.js#L8-L37

I think this solution solves the problem that eruda hides the bottom of the page where we want to inspect.

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

No branches or pull requests

3 participants