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

Implement orientation feature #136

Merged

Conversation

MarioRodriguezS
Copy link
Contributor

@MarioRodriguezS MarioRodriguezS commented Apr 3, 2024

Description

Fix #135

This pull request implements the orientation feature to editorjs-inline-image, the user is able to select a specific orientation and change the search result. The request to Unsplash is built as in the respective documentation. Also, the tests were updated and implemented.

Evidence

Screen.Recording.2024-04-03.at.3.21.29.PM.mov

@MarioRodriguezS MarioRodriguezS changed the base branch from v2.0 to v2.1 April 3, 2024 00:09
Copy link
Member

@miguelnietoa miguelnietoa left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@MarioRodriguezS well done! Some comments were added.
Don't forget to update demo.gif shown in the README file with the revamped UI 👌🏻

src/controlPanel.js Outdated Show resolved Hide resolved
src/controlPanel.js Outdated Show resolved Hide resolved
src/unsplashClient.js Outdated Show resolved Hide resolved
src/controlPanel.js Outdated Show resolved Hide resolved
@MarioRodriguezS MarioRodriguezS force-pushed the implement-orientation-feature branch from f0d7ac6 to 9ee69b2 Compare April 3, 2024 21:47
Copy link
Member

@miguelnietoa miguelnietoa left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🚀

@MarioRodriguezS MarioRodriguezS merged commit afac459 into kommitters:v2.1 Apr 3, 2024
3 checks passed
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.

2 participants