Skip to content

[FE UI] Fix some bugs about new FE UI - #4830

Merged
morningman merged 1 commit into
apache:masterfrom
px-l:update-ui
Nov 5, 2020
Merged

[FE UI] Fix some bugs about new FE UI#4830
morningman merged 1 commit into
apache:masterfrom
px-l:update-ui

Conversation

@px-l

@px-lpx-l commented Nov 2, 2020

Copy link
Copy Markdown
Contributor

Proposed changes

  1. Add a search boxer in the left tree view of Playground.
  2. Fix some visual bugs of UI.
  3. Fix bugs that link failed in QueryProfile view.
  4. Fix bugs that cookie is always invalid.
  5. Set cookie to HTTP_ONLY to make it more safe.

Types of changes

  • Bugfix (non-breaking change which fixes an issue)

1. Add a search boxer in the left tree view of Playground.
2. Fix some visual bugs of UI.
3. Fix bugs that link failed in QueryProfile view.
4. Fix bugs that cookie is always invalid.
5. Set cookie to HTTP_ONLY to make it more safe.
@morningmanmorningman self-assigned this Nov 3, 2020
@morningmanmorningman added area/fe/frontpage Issues or PRs related to the FE web page kind/fix Categorizes issue or PR as related to a bug. labels Nov 3, 2020

@morningmanmorningman left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@morningmanmorningman added the approved Indicates a PR has been approved by one committer. label Nov 3, 2020
@morningman
morningman merged commit c5b034a into apache:masterNov 5, 2020
@yangzhgyangzhg mentioned this pull request Feb 9, 2021
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

approvedIndicates a PR has been approved by one committer.area/fe/frontpageIssues or PRs related to the FE web pagekind/fixCategorizes issue or PR as related to a bug.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

@px-l@morningman@morningman-cmy