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

Feat tab view #327

Open
wants to merge 11 commits into
base: main
Choose a base branch
from
Open

Feat tab view #327

wants to merge 11 commits into from

Conversation

yashas-hm
Copy link
Contributor

@yashas-hm yashas-hm commented Mar 19, 2024

PR Description

Tab View implementation for requests

Related Issues

Checklist

  • I have gone through the contributing guide
  • I have updated my branch and synced it with project main branch before making this PR
  • I have run the tests (flutter test) and all tests are passing

Added/updated tests?

  • No, and this is why: Not sure which test cases to add
Screen.Recording.2024-03-20.at.12.58.40.AM.mov

@yashas-hm
Copy link
Contributor Author

@animator please review

@animator
Copy link
Member

The UI can be made better. It is not looking professional.

@yashas-hm
Copy link
Contributor Author

Sure will work on it.

@yashas-hm
Copy link
Contributor Author

How does this look?

Screenshot 2024-03-21 at 10 25 32 AM

@ashitaprasad
Copy link
Member

The line below the tabs is not looking good.

@yashas-hm
Copy link
Contributor Author

Screenshot 2024-03-21 at 5 26 57 PM Screenshot 2024-03-21 at 5 27 15 PM

@ashitaprasad
Copy link
Member

Looks better .. but the goal is not to make it like chrome tabs (smaller & smaller), but more like VS Code tabs that do not shrink beyond a certain width (so that it does not hamper readability) & is scrollable.
Also, looks like you have not taken the latest pull of the main branch as we added a top bar above the URL box .. see #307
So you need to redesign keeping this in mind.

@yashas-hm
Copy link
Contributor Author

The mentioned issue is not merged with main yet. But ill keep that bar in ming and redesign accordingly. Thanks for the clarity of design implementation.

@yashas-hm
Copy link
Contributor Author

VS code has an Edgy and sharp look which conflicts with the current app curved designs.

@animator
Copy link
Member

@yashas-hm Only the protocol selection dropdown is not merged. The top bar containing the name of the request and the button to rename it is already in the main branch.

@animator
Copy link
Member

You are not suggested to copy the exact design of VS Code tabs .. You should take a look at it as to how it functions and adapt it to our design.

@yashas-hm
Copy link
Contributor Author

I had removed that for testing purposes. Will include the rename button besides the tab as having a separate column below the tab view doesn't look good with the name of the request coming multiple times.

Also can the feature #307 be redesigned like the image below as it would be much better option to place the dropdown in request editor pane.

Screenshot 2024-03-21 at 6 18 52 PM

@yashas-hm
Copy link
Contributor Author

Screenshot 2024-03-21 at 6 19 10 PM

@yashas-hm
Copy link
Contributor Author

Will push changes once design finalised.

@mmjsmohit
Copy link
Contributor

Hi @yashas-hm!
The top bar has the protocol selection dropdown, the request name and a rename button as shown in the following image.
Screenshot from 2024-03-24 13-01-16

# Conflicts:
#	lib/consts.dart
@yashas-hm
Copy link
Contributor Author

I had removed that for testing purposes. Will include the rename button besides the tab as having a separate column below the tab view doesn't look good with the name of the request coming multiple times.

Also can the feature #307 be redesigned like the image below as it would be much better option to place the dropdown in request editor pane.

Screenshot 2024-03-21 at 6 18 52 PM

Hi @mmjsmohit can the dropdown be shifted in the request editor pane? As when including the tab view the name of request coming multiple times would not look pleasing according to me.

If @animator @ashitaprasad and you are ok with the double names under the tabview then I can Implement accordingly.

Current:
Screenshot 2024-03-24 at 1 17 53 PM

With topbar for protocol selection
Screenshot 2024-03-24 at 1 19 27 PM

And I am proposing something like this:
Note: This is just UI i have not implemented the dropdown
Screenshot 2024-03-24 at 1 24 19 PM

@mmjsmohit
Copy link
Contributor

Since the type of protocol used alters the rest of the UI, including that of the request and response pane, I think it is not appropriate to put it inside the request pane.

@yashas-hm
Copy link
Contributor Author

Ok then will make the changes accordingly thanks.

@yashas-hm
Copy link
Contributor Author

Final Design:

Screenshot 2024-03-24 at 1 19 27 PM

@animator
Copy link
Member

animator commented Mar 26, 2024

How is it looking on Windows .. We have the minimize maximize and close buttons on the top right.

@yashas-hm
Copy link
Contributor Author

I have a Mac Silicon so not able to run Windows as a VM but will get a friend to test and will send the screenshot.

@yashas-hm
Copy link
Contributor Author

Windows

S1

S2

@animator
Copy link
Member

The padding at the top above the tabs is not looking good.

@yashas-hm
Copy link
Contributor Author

I have added the top padding to collection panel too for windows before it was not there only in mac. We can add name of the app (APIDash and logo in the top space so that it is not empty., most desktop apps have that.

Else the tabs will be merged with the action buttons on top in windows.

@yashas-hm
Copy link
Contributor Author

Windows:
Screenshot 2024-03-27 at 4 16 04 PM

MacOS:
Screenshot 2024-03-27 at 4 16 04 PM

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

Successfully merging this pull request may close these issues.

Adding Tabs at the top of the request pane
4 participants