Skip to content

Implement Drag-and-Drop Export Functionality in Avogadro - #491

Open
vinayakjeet wants to merge 8 commits into
OpenChemistry:masterfrom
vinayakjeet:dragfrom
Open

Implement Drag-and-Drop Export Functionality in Avogadro#491
vinayakjeet wants to merge 8 commits into
OpenChemistry:masterfrom
vinayakjeet:dragfrom

Conversation

@vinayakjeet

@vinayakjeet vinayakjeet commented Mar 29, 2024

Copy link
Copy Markdown
Contributor

Pull Request #365

Introducing a Streamlined Workflow Enhancement for Avogadro: Drag-and-Drop Export Functionality!

Say goodbye to the traditional, multi-step process of exporting your molecules for use in other applications. With this update, you can simply drag a molecule from the Avogadro interface directly into another application, such as a web browser for uploading to platforms like WebMO or CalcUS.

Key Features:

  • SDF Format Support: Ideal for exporting smaller molecules, ensuring wide compatibility.
  • CJSON Format Support: For other molecules, providing flexibility in data representation.

This feature supports exporting in two major formats:

  • SDF (Simple Data Format) for smaller molecules.
  • CJSON (Chemical JSON) for others.

By implementing this feature, we aim to enhance our molecular modeling experience with an intuitive, time-saving functionality that fits seamlessly into users’ workflows.

closes #365

Comment thread avogadro/mainwindow.cpp Outdated
else
openFile(fileName);
}
if (mimeData->hasUrls()) {

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

I mean, that's okay if people drag a file to open (e.g., SDF) but it breaks drag-python-script to install: addScript(fileName) above.

Comment thread avogadro/mainwindow.cpp Outdated

else if (mimeData->hasText()) {

QString textData = mimeData->text();

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

This only looks like it affects the "drop" part .. what's the drag-from portion?

@vinayakjeet

Copy link
Copy Markdown
Contributor Author

Hi @ghutchis ,

Sorry about the mistake in PR #491. I'll fix it ASAP and make sure everything is up to par.

Thanks for your patience.

@vinayakjeet

Copy link
Copy Markdown
Contributor Author

Hi @ghutchis
Could you please share the link to the automated tests run for PRs? I'm looking to understand the testing process better. Thanks!

@vinayakjeet

Copy link
Copy Markdown
Contributor Author

I've implemented a new set of changes wherever feasible and requested a review. Through these pull requests I have created recently , I've learned more about production-ready code than I have in two years of my engineering journey. I am sincerely grateful to @ghutchis for their guidance and support. Please review and provide any necessary feedback. Thank you!!

nbehrnd and others added 8 commits April 18, 2024 04:23
Currently translated at 99.4% (166 of 167 strings)

Translation: Avogadro/avogadroapp
Translate-URL: https://hosted.weblate.org/projects/avogadro/avogadroapp/de/
Signed-off-by: Vinayakjeet Singh Karki <139736674+vinayakjeet@users.noreply.github.com>
Fixes OpenChemistry#479

Signed-off-by: Geoff Hutchison <geoff.hutchison@gmail.com>
Signed-off-by: Vinayakjeet Singh Karki <139736674+vinayakjeet@users.noreply.github.com>
Signed-off-by: Vinayakjeet Singh Karki <139736674+vinayakjeet@users.noreply.github.com>
Signed-off-by: Vinayakjeet Singh Karki <139736674+vinayakjeet@users.noreply.github.com>
Signed-off-by: Vinayakjeet Singh Karki <139736674+vinayakjeet@users.noreply.github.com>
Signed-off-by: Vinayakjeet Singh Karki <139736674+vinayakjeet@users.noreply.github.com>
Signed-off-by: Vinayakjeet Singh Karki <139736674+vinayakjeet@users.noreply.github.com>
Signed-off-by: Vinayakjeet Singh Karki <139736674+vinayakjeet@users.noreply.github.com>
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.

Support "drag from"

3 participants