-
Notifications
You must be signed in to change notification settings - Fork 112
Docs: Restructured the layout of "Understanding Tool Modes" #558
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
Conversation
✅ Deploy Preview for bejewelled-pegasus-b0ce81 ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
11af658
to
0270e37
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@REDFLAG-bugs while I found this better to read now, let's take opinion from others.
If you can attend Monday's websites meeting at 7:30 PM IST we can go through this.
Sure @sangramrath, will attend the Monday's call at 7:30 PM. |
Thank you, @sangramrath 👍 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you, @REDFLAG-bugs. Please use a tab panel.
See https://docs.layer5.io/cloud/reference/default-permissions/ for an example. |
Bonus points are available for replacing the placeholders with real screenshots, @REDFLAG-bugs. 😄 |
Bonus points?!? Where? Let me at 'em! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Try again, @REDFLAG-bugs. Be sure that you are verifying in your local build.

Yes @leecalcote, on it. |
If you've made changes and want someone to review. Document those changes (screenshot, too) and send word, @REDFLAG-bugs. |
Hello @miacycle, @sangramrath @miacycle @leecalcote - I’d appreciate it if you could review the updates made in this deploy preview : Link. |
Looks like I made a timezone conversion mistake. It's actually 5:30 PM IST. |
@REDFLAG-bugs This looks great, much clearer now! Thank you so much for your work.👍 I have a couple of minor suggestions:
And since you and @sangramrath weren't at the meeting, perhaps Sangram has some suggestions he'd like to share here as well?👀 |
Hey @zihanKuang, Thank you for reviewing. |
Hi @REDFLAG-bugs 😊, are you still working on this, or have you encountered any problems? |
Hi @zihanKuang, kindly check! |
@REDFLAG-bugs, I forgot that you were working on this and went ahead and made updates yesterday. take a look and see if you can improve upon any of those changes. |
@leecalcote So, do we don't need a tabular format to the theory? mere the the "< tab pane >" with "< tab header >" is needed? I assume, that's why a resolve conflict issue is showing. |
Notes for Reviewers
This PR fixes #541
Signed commits