Obsidian-StudiosInc / ecrire

General purpose graphical Text Editor built on the Enlightenment Foundation Libraries (EFL)
GNU General Public License v3.0
7 stars 1 forks source link

Cannot save new file #13

Closed Peter2121 closed 6 years ago

Peter2121 commented 6 years ago

When I open ecrire, type something and try to save my work - I get a corrupted dialog (see image). Efl version is 1.20.99, OS is FreeBSD 11.

err-ecrire-save

wltjr commented 6 years ago

Nice likely something changed in EFL. I will see if I can replicate. Could be something in ecrire code, but that dialog is just an elm_win_inwin with elm_fileselector. Pretty minimal code. Maybe it needs a minimum size or something, but usually windows re-size to their contents. I will take a look, thanks for the report!

wltjr commented 6 years ago

Forgot to mention, can you check Elementary Tests, if you have them installed. The FileSelector Button or Entry. Curious if they have size issues as well.

Peter2121 commented 6 years ago

No problems in Elementary Tests. Voila the errors in terminal when I reproduce the problem:


ERR<95219>:elementary lib/elementary/efl_ui_focus_manager_calc.c:1119 _logical_movement() Warning cycle detected

ERR<95219>:elementary lib/elementary/efl_ui_layout.c:1012 _efl_ui_layout_content_set() could not swallow 0x8000001faf1fef91 into part '(null)'
ERR<95219>:elementary lib/elementary/efl_ui_focus_manager_calc.c:153 node_get() Focusable 0x8000001faf1fef91 (Elm.Fileselector) not registered in manager 0x80000000ff1febbb
wltjr commented 6 years ago

Could be bad code in ecrire, but that seems like issues resulting from a change in EFL. I will file a bug in phab. Maybe they can point out the issue in ecrire code, or what ever is causing that in EFL. Strange it did not do that for me when I switched to snapshot from earlier today. But I did have other issues with themeing. I opened an issue on that. I have seen numerous issues with EFL 1.20.x Not sure what is going on... Filed several bugs today.

Peter2121 commented 6 years ago

I think that the problem is in theme file, used by ecrire

wltjr commented 6 years ago

Ecrire has no theme file :) I have considered it but not sure I like edje and not sure anyone wants to theme ecrire. At best I could see doing it for maybe mobile and desktop UI. Though I would think that to apply to E in general and not be Ecrire specific. I feel ecrire will always go with the general E theme. Not sure anyone would want a theme specific for ecrire. Other things use themes for like effects, like ephoto. But similar effects can be done in code without requiring edje.

Given problems I see with edje and entrance bad code, etc. I really dislike edje and its fraught with issues. They seem to keep changing things that break many things. Why most old themes do not work with current EFL.

Anyway I opened an upstream task on phab T6147. I would be happy to address. But I have spent time chasing my tail on other issues to find out the problem lies else where... Like this one with entrance and edje blowing up. Look at that log file attached to that task, and then what I found out was the real problem. Or this one with pinentry efl dialog positioning being off 2nd attempt, the 1st is referenced.

I have seen way to many bugs in both EFL and E to have much confidence sadly... Still have a few elm_code issues open... Ecrire is mostly a wrapper, and pretty minimal code thus few places for it to be off. Though I do know the search/replace issues in ecrire are me, and some other stuff. Thus some issues still open here, that do not have links to upstream issues.

wltjr commented 6 years ago

BTW really glad you ran into this before I made any further changes. I am planning on adding a history to the file dialog. Not sure I would have it be present for file saving, but its basically the same dialog for opening or saving. Along those lines are you having any issues on opening or just saving? Seems like it should be any use of that dialog. If its just save, that is strange.

Anyway I was thinking to add a hoversel/drop down/combobox at the top of the file dialog for recently opened/saved files. Likely the last ~10, maybe more. That likely would not cause any issues, but would be further modification which may cause issue. I will wait till we figure out this issue before proceeding on that front :)

Peter2121 commented 6 years ago

Yes, the problem is present showing "Open" dialog as well.

NuLogicSystems commented 6 years ago

That's the file selector bug that was fixed not too long ago.

wltjr commented 6 years ago

That may explain why I did not see it in latest git. I guess its fixed, and can close upstream issue.

Peter2121 commented 6 years ago

I confirm that the problem has gone after EFL updating.