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

Align dashboard menu options to the left #1335

Closed
Caelebs opened this issue Jan 20, 2021 · 15 comments · Fixed by #1338
Closed

Align dashboard menu options to the left #1335

Caelebs opened this issue Jan 20, 2021 · 15 comments · Fixed by #1338
Labels
bug Something isn't working frontend good first issue Good for newcomers

Comments

@Caelebs
Copy link

Caelebs commented Jan 20, 2021

Bug report

Describe the bug

I deployed the latest version of dashboard v2.3, and found that the Server Info in the left menu bar is not aligned, and the icons and text titles are displayed misaligned. And the content displayed in the Server Info is not very friendly. I hope that last_report_time and boot_time do not use Unix Timestamp to display. In addition, the etcd version number obtained by etcd_version is not exactly the same as the actual version number used.

Screenshots

20210120104337
20210120104404
20210120110241

System information

  • OS: CentOS Linux release 7.7.1908 (Core)
  • Browser: Chrome/78.0.3904.108
  • Version: apisix-dashboard-2.3
@Caelebs Caelebs added the bug Something isn't working label Jan 20, 2021
@juzhiyuan juzhiyuan added frontend good first issue Good for newcomers labels Jan 20, 2021
@KishaniKandasamy
Copy link
Contributor

KishaniKandasamy commented Jan 20, 2021

" Align dashboard menu options to the left".
Screenshot (34)

@KishaniKandasamy
Copy link
Contributor

but here,I think the Server Info is already in a form of human readable.
Screenshot (35)

@juzhiyuan
Copy link
Member

juzhiyuan commented Jan 20, 2021

@starsz @nic-chen Please take a look at this issue, the etcd version is incorrect according to user's feedback 🤔

@Caelebs
Copy link
Author

Caelebs commented Mar 25, 2021

image
image

I just built the dashboard of version 2.5, and found that this item is still not aligned. Is the code merging missing?
Browser version: Google Chrome | 87.0.4280.66 (正式版本) (64 位) (cohort: 87_66_Win)

@juzhiyuan @KishaniKandasamy

@juzhiyuan
Copy link
Member

🤔 not sure, let me reopen this.

@juzhiyuan juzhiyuan reopened this Mar 25, 2021
@nic-chen
Copy link
Member

@Caelebs
Is there still a problem with the wrong etcd version?

@juzhiyuan
Copy link
Member

@nic-chen Hi, this is is a UI issue :)

@nic-chen nic-chen added this to the 2.6 milestone Apr 7, 2021
@nic-chen nic-chen modified the milestones: 2.6, 2.7 Apr 19, 2021
@Caelebs
Copy link
Author

Caelebs commented Jun 16, 2021

I just deployed the latest v2.7 version of the dashboard, and found that this does not seem to be fixed? Is it missed?
image
image

@juzhiyuan

@juzhiyuan
Copy link
Member

Not sure, please take a look :)

@nic-chen nic-chen removed this from the 2.7 milestone Jun 16, 2021
@iamayushdas
Copy link
Contributor

Having a look to it

@iamayushdas
Copy link
Contributor

iamayushdas commented Jun 17, 2021

@Caelebs @juzhiyuan @liuxiran its not fixed yet, i checked in my env too

and according to my inspection the reason is ant-info-circle icon is small(14x14) where as others are of (16x16)

@juzhiyuan
Copy link
Member

good, keep it as Good first issue

@liuxiran
Copy link
Contributor

close via #2051 , feel free to reopen it when there are any questions, thanks

@Caelebs
Copy link
Author

Caelebs commented Jun 9, 2023

I deployed the latest version of apisix and dashboard a few days ago, and this issue seems to have reappeared.
version:
apisix:3.3.0
apisix-dashboard:3.0.1
Snipaste_2023-06-06_19-24-36
Snipaste_2023-06-06_19-24-04

@Caelebs
Copy link
Author

Caelebs commented Jun 9, 2023

@juzhiyuan

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working frontend good first issue Good for newcomers
Projects
None yet
Development

Successfully merging a pull request may close this issue.

6 participants