Skip to content

Renamer prerelease styles#646

Open
madebyisaacr wants to merge 4 commits into
mainfrom
update-renamer-icons
Open

Renamer prerelease styles#646
madebyisaacr wants to merge 4 commits into
mainfrom
update-renamer-icons

Conversation

@madebyisaacr
Copy link
Copy Markdown
Collaborator

Description

This pull request adds prerelease styles to the Renamer plugin, including new icons and tabs design.

Screenshots: https://www.notion.so/framer/365adf6e8c968017adafebc517001a38

Changelog

  • Updated icons
  • Added icon for breakpoint layers

Testing

  • Verify styles are correct with experiment on and off

@madebyisaacr madebyisaacr requested a review from Nick-Lucas May 19, 2026 17:10
@github-actions github-actions Bot added the Auto submit to Marketplace on merge Submits the plugin to the marketplace after merging label May 19, 2026
icon = (
<svg
xmlns="http://www.w3.org/2000/svg"
width="11"
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.

I expect cases like this are why we had 12 set as the parent element width/height (maybe the icon had a large perceived size so we dropped by 1?) but we seem to have lost that in the change

This file may be worth keeping that behaviour unless we can demonstrate it was a mistake before

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.

Nevermind I see you just moved it to the class which makes sense: https://github.com/framer/plugins/pull/646/changes#r3272172807

</button>
))}
</div>
</div>
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.

these could probably be conditions over the className probably rather than duplicating the structure, we already have cx which is designed for this need

padding: 15px;
}

[data-framer-styles="prerelease"] .container {
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.

an alternative for the tabs could be to use this pattern instead, so that the new classes apply when data-framer-styles is set in the root

.icon {
flex-shrink: 0;
width: 12px;
height: 12px;
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.

Ah I see now, ignore my other comment then

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Auto submit to Marketplace on merge Submits the plugin to the marketplace after merging

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants