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

Enhancement: Case study page UI Improvement #685

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

TenzDelek
Copy link

  • Please check if the PR fulfills these requirements
  • The commit message follows our guidelines
  • Tests for the changes have been added (for bug fixes / features)
  • Docs have been added / updated (for bug fixes / features)

Which issue(s) this PR fixes:

Fixes #

  • What kind of change does this PR introduce? (Bug fix, feature, docs update, ...)

UI changes

  • What is the current behavior? (You can also link to an open issue here)

old user interface

  • What is the new behavior (if this is a feature change)?

redesign and implement new UI with mobile responsiveness and a clean code

  • Does this PR introduce a breaking change? (What changes might users need to make in their application due to this PR?)

no

  • Other information:

Screenshots:

Screenshot 2025-02-12 at 9 54 18 PM Screenshot 2025-02-12 at 9 54 33 PM

In Mobile view

smallerscreen

@kubeedge-bot
Copy link
Collaborator

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by:
To complete the pull request process, please assign fisherxu after the PR has been reviewed.
You can assign the PR to them by writing /assign @fisherxu in a comment when ready.

The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@kubeedge-bot
Copy link
Collaborator

Welcome @TenzDelek! It looks like this is your first PR to kubeedge/website 🎉

@kubeedge-bot kubeedge-bot added the size/L Denotes a PR that changes 100-499 lines, ignoring generated files. label Feb 12, 2025
Signed-off-by: TenzDelek <[email protected]>
@TenzDelek
Copy link
Author

PTAL @Shelley-BaoYue, As this page was not mentioned in the lfx task, i thought of creating it nonetheless for the UI improvement. can you please take a look at it . thank you

/assign @Shelley-BaoYue

@harshita9104
Copy link

Hello @TenzDelek ,
I have already worked on this issue and created a PR for the same (#674)
which was also reviewed by @Shelley-BaoYue

@TenzDelek
Copy link
Author

Hello @TenzDelek , I have already worked on this issue and created a PR for the same (#674) which was also reviewed by @Shelley-BaoYue

@harshita9104 , I only see mobile responsiveness in your pr. this pr is for overall UI improvement. including the responsiveness.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
size/L Denotes a PR that changes 100-499 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants