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

Modal for posting new posts #86

Closed
wants to merge 4 commits into from
Closed

Modal for posting new posts #86

wants to merge 4 commits into from

Conversation

hellcp
Copy link
Member

@hellcp hellcp commented Jun 1, 2018

peek 2018-06-01 10-39

@mkljczk
Copy link
Contributor

mkljczk commented Jun 1, 2018

Seems like it requires JS, but “javascript is just a progressive enhancement in PixelFed”… Using for previous card is a good idea…

@hellcp
Copy link
Member Author

hellcp commented Jun 1, 2018

@m4sk1n it's enabled through js too, so it will only change to modal mode if you got js ;)

@dansup dansup self-assigned this Jun 1, 2018
@dansup dansup added refactoring 🤔 In-Depth Requires more investigation labels Jun 1, 2018
@dansup
Copy link
Member

dansup commented Jun 3, 2018

@hellcp Nice work! Would you mind updating this PR to change the resources/views/timeline/partial/new-form.blade.php view instead of the personal/public timeline views?

@dansup dansup added question 👀 UI An issue with the interface / frontend 💼 WIP This is currently being worked on and removed 🤔 In-Depth Requires more investigation labels Jun 3, 2018
@hellcp hellcp changed the base branch from dev to improve-user-profile June 3, 2018 11:17
@hellcp hellcp changed the base branch from improve-user-profile to frontend-ui-refactor June 3, 2018 11:17
@hellcp
Copy link
Member Author

hellcp commented Jun 3, 2018

Will continue in other PR

@hellcp hellcp closed this Jun 3, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
👀 UI An issue with the interface / frontend 💼 WIP This is currently being worked on
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants