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

Page Title #817

Merged
merged 3 commits into from
Aug 19, 2024
Merged

Conversation

ralikio
Copy link
Member

@ralikio ralikio commented Aug 19, 2024

Description

Changes proposed in this pull request:

  • added title attribute to Layout component to show information about which view is currently active,
  • forced display property to be set to inline instead of grid to allow title block placement before the Secrets view,
  • added a 1rem padding to put some space between the title and the top of an ObjectPage,
  • tidying imports.

Related issue(s)

#442

Added title attribute to Layout component to show information about which view is currently active.
Forced display property to be set to inline instead of grid to allow title `block` placement before the `Secrets` view.

Added a 1rem padding to put some space between the title and the top of an ObjectPage.
@ralikio ralikio added the kind/enhancement Categorizes issue or PR as related to modifying or improving an existing feature label Aug 19, 2024
@ralikio ralikio requested a review from a team as a code owner August 19, 2024 11:11
@kyma-bot kyma-bot added cla: yes Indicates the PR's author has signed the CLA. size/L Denotes a PR that changes 100-499 lines, ignoring generated files. labels Aug 19, 2024
@ralikio ralikio changed the title Features/route title Page Title Aug 19, 2024
@kyma-bot kyma-bot added the lgtm Looks good to me! label Aug 19, 2024
@ralikio ralikio merged commit 3ec54fb into kyma-project:sm-integration Aug 19, 2024
14 checks passed
szwedm pushed a commit that referenced this pull request Aug 21, 2024
* Setting Page Title from Within Route Component

Added title attribute to Layout component to show information about which view is currently active.

* Removing Unnecessary Imports

* CSS Update for Object Page

Forced display property to be set to inline instead of grid to allow title `block` placement before the `Secrets` view.

Added a 1rem padding to put some space between the title and the top of an ObjectPage.
kyma-gopher-bot pushed a commit that referenced this pull request Aug 22, 2024
* Setting Page Title from Within Route Component

Added title attribute to Layout component to show information about which view is currently active.

* Removing Unnecessary Imports

* CSS Update for Object Page

Forced display property to be set to inline instead of grid to allow title `block` placement before the `Secrets` view.

Added a 1rem padding to put some space between the title and the top of an ObjectPage.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cla: yes Indicates the PR's author has signed the CLA. kind/enhancement Categorizes issue or PR as related to modifying or improving an existing feature lgtm Looks good to me! 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.

3 participants