[go: nahoru, domu]

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

perf: pager component display logical optimization #67

Merged
merged 7 commits into from
May 28, 2022
Merged

perf: pager component display logical optimization #67

merged 7 commits into from
May 28, 2022

Conversation

rainbowatcher
Copy link
Contributor
@rainbowatcher rainbowatcher commented May 24, 2022

Related ISSUE

Type Of Change

  • 🐞 Bug fix (non-breaking change which fixes an issue)
  • ✨ New feature (non-breaking change which adds functionality)
  • 📝 Document (This change requires a documentation update)
  • 🎨 Theme style (Theme style beautification)
  • ⚠ Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • 🔨 Workflow (Workflow changes)

Clear Describe

pager component display logical optimization

Description

  1. according the habit to most people,
    put button "previous" to left and the button "next" to right.
  2. use grid style on Pager component.
  3. add a little animation to these button.
  4. post title inside the button limited to 2 lines.
  5. add markdown sub & sup syntax support.
  6. add the docs about.

Test Case

Scrrenshots

This is what the Pager component display in my blog. When the title of one of the buttons occupies two lines and the other only occupies one line, the display is abnormal.
before-blog2

and after will be like this
longtitle-mobile

and below is the screenshot of pc and pad

longtitle-pc

longtitle-pad

the two markdown file is for test.

1. according the habit to most people,
   put button "previous" to left and the button "next" to right.
2. use grid style on Pager component.
3. add a little animation to these button.
4. post title inside the button limited to 2 lines.
@vercel
Copy link
vercel bot commented May 24, 2022

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Updated
v2-vuepress-theme-gungnir ✅ Ready (Inspect) Visit Preview May 27, 2022 at 6:48AM (UTC)
vuepress-theme-gungnir ❌ Failed (Inspect) May 27, 2022 at 6:48AM (UTC)

@rainbowatcher
Copy link
Contributor Author

@Renovamen 辛苦帮我review一下🍺

docs/docs/md-enhance/others.md Show resolved Hide resolved
docs/docs/plugins/md-plus.md Show resolved Hide resolved
docs/docs/plugins/md-plus.md Outdated Show resolved Hide resolved
docs/docs/plugins/md-plus.md Outdated Show resolved Hide resolved
docs/zh/docs/plugins/md-plus.md Outdated Show resolved Hide resolved
docs/zh/docs/plugins/md-plus.md Outdated Show resolved Hide resolved
packages/plugins/md-plus/package.json Outdated Show resolved Hide resolved
docs/posts/2022-05-24.md Show resolved Hide resolved
@Renovamen Renovamen merged commit c152796 into Renovamen:main May 28, 2022
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.

None yet

2 participants